RFR: 8315844: $LSB_RELEASE is not defined before use
Julian Waters
jwaters at openjdk.org
Thu Apr 3 14:41:50 UTC 2025
On Thu, 3 Apr 2025 10:04:59 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
> When running configure on WSL, the following error message is printed, instead of proper Linux environment info:
>
>
> /build/.configure-support/generated-configure.sh: line 13102: -d: command not found
>
>
> This is due to us calling `$LSB_RELEASE -d`, but LSB_RELEASE is not yet defined at that point in time. Fix this by moving the LSB_RELEASE check forward.
Looks good, just 1 question
make/autoconf/basic_tools.m4 line 109:
> 107: UTIL_LOOKUP_PROGS(WHOAMI, whoami)
> 108:
> 109: # Tools only needed on some platforms
Comment still relevant? Or is it needed on all platforms?
-------------
Marked as reviewed by jwaters (Committer).
PR Review: https://git.openjdk.org/jdk/pull/24409#pullrequestreview-2740142274
PR Review Comment: https://git.openjdk.org/jdk/pull/24409#discussion_r2027156433
More information about the build-dev
mailing list