RFR: JDK-8312467: relax the builddir check in make/autoconf/basic.m4

Matthias Baesken mbaesken at openjdk.org
Wed Jul 26 07:21:13 UTC 2023


On Tue, 25 Jul 2023 12:00:56 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

> In case of issues in the configure step, we run into error messages like this :
> 
> configure: Current directory is /openjdk/jdk_4/build_mymachine.
> configure: Since this is not the source root, configure will output the configuration here
> configure: (as opposed to creating a configuration in <src_root>/build/<conf-name>).
> configure: However, this directory is not empty. This is not allowed, since it could
> configure: seriously mess up just about everything.
> configure: Try 'cd /mysrcdir/jdk' and restart configure
> configure: (or create a new empty directory and cd to it).
> configure: error: Will not continue creating configuration in /openjdk/jdk_4/build_mymachine
> configure exiting with result code 1
> 
> Probably the check could be relaxed; it tests for 'if test ! -e "$OUTPUTDIR/spec.gmk"; then ...'
> but the non-existance of spec.gmk does not mean we are not in a build directory.
> We could additionally check for e.g. $OUTPUTDIR/configure-support/config.log or something related to check if it is a build directory .

Hi Christoph, I adjusted the check.
Additionally I adjusted the output in the failure case, the 'bad' files causing the check to fail are printed now .

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

PR Comment: https://git.openjdk.org/jdk/pull/15017#issuecomment-1651114791



More information about the build-dev mailing list