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

Bug 306591

Summary: dev-php/roadsend-php doesn't respect CFLAGS/LDFLAGS, prestripped files
Product: Gentoo Linux Reporter: Doktor Notor <notordoktor>
Component: New packagesAssignee: PHP Bugs <php-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: flameeyes, scheme
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
URL: http://code.roadsend.com/pcc/ticket/3495 http://code.roadsend.com/pcc/ticket/3531 http://code.roadsend.com/pcc/ticket/3523
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 331933    
Attachments: roadsend-php-2.9.9_p1-flags.patch
roadsend-php-2.9.9_p1.ebuild.diff
roadsend-php-2.9.9_p1.ebuild.diff
roadsend-php-2.9.9_p1.ebuild.diff

Description Doktor Notor 2010-02-24 01:33:58 UTC
Wrt Bug 273210 Comment #7 -patch attached, this was a huge PITA and really needs to get sorted out upstream. I might have missed something anyway. libphp-pdo_u-3.2b.so seems unfixable, probably pcc compiler bug, won't waste more time on this.

Wrt the prestripped files, well - bigloo itself seems to strip those and I can't find any command-line option to disable such behaviour, so... NFC, sorry.
Comment 1 Doktor Notor 2010-02-24 01:34:59 UTC
Created attachment 220905 [details, diff]
roadsend-php-2.9.9_p1-flags.patch
Comment 2 Doktor Notor 2010-02-24 01:37:16 UTC
Created attachment 220907 [details]
roadsend-php-2.9.9_p1.ebuild.diff

As said, fugly... you've been warned :P
Comment 3 Doktor Notor 2010-02-24 02:58:04 UTC
Created attachment 220911 [details]
roadsend-php-2.9.9_p1.ebuild.diff

Sorry, fixed wrong sed in previous one.
Comment 4 Doktor Notor 2010-03-02 12:00:32 UTC
Created attachment 221787 [details, diff]
roadsend-php-2.9.9_p1.ebuild.diff

After a couple of emails with upstream (Shannon Weyrick), it seems muting the remaining QA warnings until bigloo is gone from this package is the way to go here, see the comment in the patch.
Comment 5 Doktor Notor 2010-04-26 09:44:41 UTC
*** Bug 315723 has been marked as a duplicate of this bug. ***
Comment 6 Matti Bickel (RETIRED) gentoo-dev 2010-06-18 22:24:51 UTC
Thanks for the patch and sorry for responding so late. The patch looks good to me. I'll recheck it tomorrow and if I don't find some 'rm -rf /' in there, I'll push it into the tree.
Comment 7 Matti Bickel (RETIRED) gentoo-dev 2010-08-09 22:40:02 UTC
Well, "tomorrow" it isn't, but nevertheless - committed your work to CVS. Thanks a bunch, that patch is big!

I don't use roadsend, but let's see if I can track upstream for those changes. It seems the most of the mess will fix itself when 3.0 comes out...