[PATCH] Enable jtreg tests to run on WSL (Windows Subsystem for Linux)
Andrew Luo
andrewluotechnologies at outlook.com
Tue Jan 8 18:30:59 UTC 2019
Hi Jon,
Thanks, looks good to me.
I guess that a path is a subset of path list, although I’m not sure why Microsoft gives them separate suffixes…
In Windows, technically the correct case is “Path”. However often times people use PATH (or path), and Linux environment variables are case sensitive, while Windows environment variables are not, so if someone set Linux PATH, path, and Path, we don’t want any of them to be translated to Windows, as I believe any of them would override the Windows path.
I didn’t see any blank env elements, the .isEmpty() check must have been by habit.
I did test your changes on my WSL environment and everything compiles/works.
Thanks,
-Andrew
From: Jonathan Gibbons <jonathan.gibbons at oracle.com>
Sent: Monday, January 7, 2019 4:26 PM
To: Andrew Luo <andrewluotechnologies at outlook.com>; code-tools-dev at openjdk.java.net
Subject: Re: [PATCH] Enable jtreg tests to run on WSL (Windows Subsystem for Linux)
Andrew,
I've put out an updated webrev, that is somewhat simpler than your version, although it should be functionally equivalent. In particular, it has a much smaller impact on the non-WSL code.
Some questions:
* Why is it important to distinguish "path" and "path list"? Isn't a "path" just a "path list" with a single entry? In other words, can we always use suffix `/l` for a list of one-or-more paths?
* Is it important to ignore case when checking for "PATH"? Would the Windows environment ever contain an env variable "path"?
* The env should never have entries in it with "blank" names; was it just defensive programming on your part to check for blank names and ignore them, or did you see them in practice?
* I'm not set up here right now to be able to test with WSL; can you try building and testing this version of jtreg?
-- Jon
Webrev: http://cr.openjdk.java.net/~jjg/7902357/webrev.02/
More information about the code-tools-dev
mailing list