On Mon, 7 Nov 2022 00:28:13 GMT, Mike Duigou <mdui...@openjdk.org> wrote:
>> The OpenJDK build system does not support building when the source code >> resides on a path that contains a space. This requirement is documented in >> the build instructions but not enforced by the configure script. >> >> This change adds an explicit checks to the wrapper `configure` script that >> fail the build if the source code to be built is located in a directory >> who's path contains a space character or the build path cannot be determined. >> >> Includes some idiom modernization and shell scripting best practices changes. > > Mike Duigou has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by the merge/rebase. The pull request contains one additional > commit since the last revision: > > 8294549: configure script should detect unsupported path > > The OpenJDK build system does not support building when the source code > resides on a path that contains a space. This requirement is documented in > the > build instructions but not enforced by the configure script. > > This change adds an explicit checks to the wrapper `configure` script that > fail > the build if the source code to be built is located in a directory who's > path > contains a space character or the build path cannot be determined. configure line 34: > 32: > 33: if [ -z "${this_script_dir}" ]; then > 34: echo "# Could not determine location of configure script" Why the initial `#`? ------------- PR: https://git.openjdk.org/jdk/pull/10477