Bug 260847 – media-video/shrip fails to build with _FORTIFY_SOURCE=2

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Diego Elio Pettenò (RETIRED) 2009-03-02 11:07:40 UTC]

Check attached build log and http://blog.flameeyes.eu/2009/02/25/future-proof-your-code-dont-use-werror .

[Comment 1 Diego Elio Pettenò (RETIRED) 2009-03-02 11:08:02 UTC]

Created attachment 183668 [details]
Build log

[Comment 2 Magnus Granberg 2009-03-06 00:54:19 UTC]

Patch for the -Werror is in bug #251367

[Comment 3 Olivier Rolland 2009-07-22 19:03:24 UTC]

Created attachment 198824 [details, diff]
shrip-0.5.0-wur.patch

[Comment 4 Olivier Rolland 2009-07-22 19:05:11 UTC]

Created attachment 198826 [details, diff]
shrip-0.5.0-werror.patch

This patch replaces -Werror with -D_FORTIFY_SOURCE=2.

[Comment 5 Gordon Malm (RETIRED) 2009-07-22 19:16:16 UTC]

Unused result patch added and -Werror removed in =media-video/shrip-0.5.0-r1. 
All previous versions mentioned here no longer apply.  Closing bug as FIXED.

[Comment 6 Gordon Malm (RETIRED) 2009-07-22 19:19:55 UTC]

Olivier,

The shrip-0.5.0-wur.patch was added in =media-video/shrip-0.5.0-r1, -Werror simply removed.  Adding -D_FORTIFY_SOURCE=2 is not necessary, =sys-devel/gcc-4.3.3-r2 already adds it.

[Comment 7 Olivier Rolland 2009-07-22 19:55:50 UTC]

But I don't use gcc-4.3.3 ;-)

[Comment 8 Gordon Malm (RETIRED) 2009-07-22 20:31:58 UTC]

It doesn't matter.  It's not the correct place to add it in Gentoo and we're not doing it that way for any other packages in the tree.