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

Mike Duigou mduigou at openjdk.org
Sat Dec 3 22:06:04 UTC 2022


On Mon, 7 Nov 2022 12:27:03 GMT, Magnus Ihse Bursie <ihse 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.
>
> configure line 34:
> 
>> 32: 
>> 33: if [ -z "${this_script_dir}" ]; then
>> 34: 	echo "# Could not determine location of configure script"
> 
> Why the initial `#`?

Just habit. Some style guides for shell commands suggest putting a "#" before error output.

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

PR: https://git.openjdk.org/jdk/pull/10477



More information about the build-dev mailing list