#1181 Compiler flags updates
Opened by oturpe. Modified
oturpe/packaging-committee compiler-flags-fixes  into  master

Download 1181.patch
  • Add named anchor to a section in RPMMacros.adoc.
    The index page already attempts to link to such anchor,
    but fails because it is not defined.

  • Update the example results for optflags and build_ldflags
    to match Fedora 36 configuration,
    up from Fedora 28.

  • Remove a paragraph about allowing adding some security related build
    flags.
    This is a reference to the PIE section immediately after,
    but at some point, PIE became the default and that section does not
    suggest adding any flags any more.

I wonder whether we should actually list the actually evaluated macros in the documentation? That tends to become outdated fast (as we see here). Maybe just saying "use rpm --eval %_build_cflags on the target to see the compiler flags that are in effect" would be more future-proof?

I wonder whether we should actually list the actually evaluated macros in the documentation? That tends to become outdated fast (as we see here). Maybe just saying "use rpm --eval %_build_cflags on the target to see the compiler flags that are in effect" would be more future-proof?

I was wondering about that also when I saw the listing.
One downside I can think is that checking what the flags are requires a Fedora installation.
Probably that is not a real issue,
I support not many people package for Fedora using some other environment.
I will wait a bit to see if somebody has a different opinion,
if not, I will adjust this pull request accordingly.

Metadata