[PATCH] Enable jtreg tests to run on WSL (Windows Subsystem for Linux)

Jonathan Gibbons jonathan.gibbons at oracle.com
Tue Jan 8 21:21:26 UTC 2019


Andrew,

One more enhancement, in webrev.03. The new command line option is 
rejected if jtreg is not running on Windows.

-- Jon


On 01/08/2019 10:30 AM, Andrew Luo wrote:
>
> 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/ 
> <http://cr.openjdk.java.net/%7Ejjg/7902357/webrev.02/>
>



More information about the code-tools-dev mailing list