RFR: 8294549: configure script should detect unsupported path [v2]

Magnus Ihse Bursie ihse at openjdk.org
Fri Dec 2 13:17:08 UTC 2022


On Mon, 7 Nov 2022 00:38:23 GMT, Mike Duigou <mduigou at 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



More information about the build-dev mailing list