RFR: 8312591: GCC 6 build failure after JDK-8280982 [v2]
Aleksey Shipilev
shade at openjdk.org
Fri Jul 28 07:46:51 UTC 2023
On Thu, 27 Jul 2023 19:37:33 GMT, Phil Race <prr at openjdk.org> wrote:
>> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Disable the warnings instead
>
> make/modules/java.desktop/lib/Awt2dLibraries.gmk line 248:
>
>> 246: DISABLED_WARNINGS_gcc_OGLBufImgOps.c := format-nonliteral, \
>> 247: DISABLED_WARNINGS_gcc_OGLPaints.c := format-nonliteral, \
>> 248: DISABLED_WARNINGS_gcc_screencast_pipewire.c := undef, \
>
> At least ignoring it just for the specific compilation units is not too bad.
>
> I'll approve but I'd like to see a comment here as to why its needed - ie just for gcc6 so people have an idea when we might be able to remove it and know not to remove it because it doesn't matter for the current compiler version.
>
> I've still asked upstream to consider applying your first version so that we'd not need this although perhaps it won't matter for long .. once no one cares about gcc6 because we need to move on for other reasons.
There is no good way to put the comment straight into this block, AFAICS. Can we settle on repository history to tell future investigations why this was added?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14995#discussion_r1277221303
More information about the client-libs-dev
mailing list