RFR: JDK-8323008: filter out any -std* flags added by autoconf from CC/CXX [v3]
Christoph Langer
clanger at openjdk.org
Thu Jan 11 13:50:26 UTC 2024
On Thu, 11 Jan 2024 11:19:01 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
>> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
>>
>> adjust COPYRIGHT year
>
> make/autoconf/toolchain.m4 line 395:
>
>> 393: # filter out some unwanted additions autoconf may add to CXX; we saw this on macOS with autoconf 2.72
>> 394: UTIL_GET_NON_MATCHING_VALUES(cxx_filtered, $CXX, -std=c++11 -std=gnu++11)
>> 395: if test "x$cxx_filtered" != x; then
>
> Why this test? If CXX is empty, then xcc_filtered will be empty too, right? And if CXX is exactly `-std=c++11`, then this test will render cxx_filter empty too, which will not change CXX -- which I believe is not what you want?
I agree with @magicus. It should work when you then unconditionally assign cxx_filtered to CXX after the call.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17301#discussion_r1448892744
More information about the build-dev
mailing list