On Thu, Jan 12, 2023 at 6:38 PM Jakub Jelinek <jakub(a)redhat.com> wrote:
But at least you don't need both -U_FORTIFY_SOURCE and
-Wp,-U_FORTIFY_SOURCE, one of them is enough. And the latter I think
better gets through libtool and other tools; especially if you put it
into a single -Wp, option together with the redefinition...
Uhmm, I could have sworn that I had needed -Wp,-U... for the -Wp-D...
and -U for the -D when I was fixing the test builds but you're right,
simply -Wp,-U_FORTIFY_SOURCE -Wp,-D_FORTIFY_SOURCE=3 appears to work
for both -D_FORTIFY_SOURCE=2 as well as -Wp,-D_FORTIFY_SOURCE=2. I'll
test this some more and post a PR for redhat-rpm-config.
Thanks,
Sid