On Fri, 4 Dec 2020 14:22:45 GMT, Jorn Vernee <jver...@openjdk.org> wrote:

>> @JornVernee I have now pushed a fix that will make `fixpath import` ignore 
>> invalid or unknown paths be ignored. Please try again with it.
>
> @magicus Thanks! But, now I get the same error as the GH action: 
> https://github.com/magicus/openjdk-sandbox/runs/1498822974#step:11:80
> 
> Looks like `-a` is not supported in my shell. I can make the error go away 
> locally with:
> 
> diff --git a/make/scripts/fixpath.sh b/make/scripts/fixpath.sh
> index ae5ca1f8a28..c7737a23eb8 100644
> --- a/make/scripts/fixpath.sh
> +++ b/make/scripts/fixpath.sh
> @@ -207,7 +207,7 @@ function import_command_line() {
>      if ! [[ $arg =~ ^" "+$ ]]; then
>        import_path "$arg"
> 
> -      if [[ "$result" != "" -a "$imported" = "" ]]; then
> +      if [[ "$result" != "" ]] && [[ "$imported" = "" ]]; then
>          imported="$result"
>        else
>          imported="$imported:$result"
> But I'm not sure that does the same thing.
> 
> After that, I can confirm that both path issues with Cygwin are resolved 
> (`usr\local\bin` and paths with spaces don't cause a hard failure anymore).
> 
> ---
> 
> The issue with WSL still remains though:
> 
> configure: Found potential Boot JDK using configure arguments
> configure: Potential Boot JDK found at /mnt/c/Program Files/Java/jdk-15 is 
> not a working JDK; ignoring
> configure: Output from java -version was: 
> /mnt/h/cygwin64/home/Jorn/cygwin-projects-new/git-jdk2/build/.configure-support/generated-configure.sh:
>  line 57167: -version: command not found
> configure: error: The path given by --with-boot-jdk does not contain a valid 
> Boot JDK
> configure exiting with result code 1
> 
> Line 57167 in generated-configure.sh is this:
> 
>     # Now join together the path and the arguments once again
>     new_complete="$prefix$new_path$arguments"

Ok, after looking at `set -x` output WSL seems to have a problem with the space 
in the path `/mnt/c/Program Files/Java/jdk-15` e.g. if I move the jdk to 
`/mnt/h/jdk-15` it works. Might be a missing fixup somewhere? (FWIW, that 
Program Files dir does have an 8dot3 name).

With that I can build `images` on both Cygwin and WSL. Build times are about 29 
mins and 18 mins respectively.

-------------

PR: https://git.openjdk.java.net/jdk/pull/1597

Reply via email to