On Mon, 7 Nov 2022 00:38:23 GMT, Mike Duigou <mdui...@openjdk.org> wrote:
>> Mike Duigou has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains one commit: >> >> 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. > > I have removed the path checking from the top level configure script but > changes are still required in the `configure` wrapper to correctly pass a > potentially bad path for the real configure script. I considered adding the > path checking to `basic.m4` in `BASIC_SETUP_PATHS` but could not understand > why `UTIL_FIXUP_PATH(TOPDIR)` doesn't already reject the bad path like it > promises. (I was also reminded just how loathsome autoconf is as a > languageāall the worst parts of shell with even more cruft and weirdness). @bondolo Do you want me to take over this bug, add you as a contributor, and get it committed? Otherwise I think if you incorporate my patch above, you're good to go. ------------- PR: https://git.openjdk.org/jdk/pull/10477