Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 260847

Summary: media-video/shrip fails to build with _FORTIFY_SOURCE=2
Product: Gentoo Linux Reporter: Diego Elio Pettenò (RETIRED) <flameeyes>
Component: New packagesAssignee: Olivier Rolland <billl>
Severity: normal CC: gengor, media-video
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Package list:
Runtime testing required: ---
Bug Depends on: 251367    
Bug Blocks: 260867, 259417    
Attachments: Build log

Description Diego Elio Pettenò (RETIRED) gentoo-dev 2009-03-02 11:07:40 UTC
Check attached build log and .
Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev 2009-03-02 11:08:02 UTC
Created attachment 183668 [details]
Build log
Comment 2 Magnus Granberg gentoo-dev 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]
Comment 4 Olivier Rolland 2009-07-22 19:05:11 UTC
Created attachment 198826 [details, diff]

This patch replaces -Werror with -D_FORTIFY_SOURCE=2.
Comment 5 Gordon Malm (RETIRED) gentoo-dev 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) gentoo-dev 2009-07-22 19:19:55 UTC

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) gentoo-dev 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.