Re: C-specific compiler parameters (was: Update on Changes/PortingToModernC)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, May 10, 2023 at 12:09:10AM +0200, Björn Persson wrote:
> Florian Weimer wrote:
> > I am going to explore a way to land -Werror=implicit-int
> > -Werror=implicit-function-declaration among the default compiler flags.
> > There's a bit of an issue because the C++ front end warns on
> > those flags, so we need another -specs= kludge that is incompatible with
> > Clang.
> 
> It sounds like those parameters should be added only to CFLAGS, not to
> CXXFLAGS.
> 
> __global_compiler_flags already contains things that cause warnings
> from the Ada and Fortran compilers. The Ada packages get the warning
> “'-Werror=' argument '-Werror=format-security' is not valid for Ada”
> over and over. It doesn't break any builds but it's annoying noise in
> the build logs.

GCC 13 has a solution for that, one can add
-Wno-complain-wrong-lang
to
-Werror=format-security
etc. and avoid such warnings (that some compiler option is only appropriate
for a subset of GCC languages and it is compiling some other language).

	Jakub
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux