[Rev 01] RFR: 8241476: Linux build warning issued on updated compilers (Ubuntu 18.04.4 / 20.04)
Kevin Rushforth
kcr at openjdk.java.net
Thu Mar 26 11:23:20 UTC 2020
On Thu, 26 Mar 2020 08:33:29 GMT, Johan Vos <jvos at openjdk.org> wrote:
>> Like this?
>>
>> I think all C builds are covered. Gstreamer seems to have a Makefile with the flags. Not sure about libxml and libxlst
>> inside javafx.web.
>
> I think it's better to split the flags between CFLAGS and CPPFLAGS (as is done in OpenJDK). The
> `-Werror=implicit-function-declaration` is extremely useful for C files, and we don't want to risk that this somehow
> got lost. If we use CFLAGS and CPPFLAGS, we can pass the latter as CXXFLAGS to cmake, so that should be fairly easy.
I took a look yesterday and came to the same conclusion that what we really want are separate C and C++ flags. For now,
the only difference would be the presence or absence of `-Werror=implicit-function-declaration`, but it would allow for
other differences in the future if needed.
-------------
PR: https://git.openjdk.java.net/jfx/pull/150
More information about the openjfx-dev
mailing list