RFR: 8221907: make reconfigure broken with "build/jmh/jars does not exist or is not a directory"
Erik Joelsson
erik.joelsson at oracle.com
Wed Apr 3 15:33:52 UTC 2019
Hello Jie,
This issue applies not only to --with-jmh, but to any configure
parameter given with a relative path. I think the proper fix would be to
record the current working directory when configure is launched and cd
to that directory when running reconfigure. Here is my suggested patch:
http://cr.openjdk.java.net/~erikj/8221907/webrev.01/index.html
The relevant parts are exporting the variable from configure and using
it in Init.gmk. The rest is just renaming the variable since CURDIR
would clash with the pre defined make variable CURDIR.
/Erik
On 2019-04-03 05:28, Jie Fu wrote:
> Hi all,
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8221907
>
> For more info (e.g. the symptom & how to reproduce), please see the JBS.
>
> It can be fixed by
> -----------------------------------------
> diff -r 3326be37cd9a make/autoconf/lib-tests.m4
> --- a/make/autoconf/lib-tests.m4 Tue Apr 02 17:27:48 2019 -0700
> +++ b/make/autoconf/lib-tests.m4 Wed Apr 03 19:56:24 2019 +0800
> @@ -73,6 +73,10 @@
> else
> # Path specified
> JMH_HOME="$with_jmh"
> + if test "x${JMH_HOME:0:1}" != x/; then
> + JMH_HOME="$TOPDIR/$JMH_HOME"
> + fi
> +
> if test ! -d [$JMH_HOME]; then
> AC_MSG_RESULT([no, error])
> AC_MSG_ERROR([$JMH_HOME does not exist or is not a directory])
> -----------------------------------------
>
> Could you please review it?
> Thanks a lot.
>
> Best regards,
> Jie
>
>
More information about the build-dev
mailing list