Bug 306591

Summary:	dev-php/roadsend-php doesn't respect CFLAGS/LDFLAGS, prestripped files
Product:	Gentoo Linux	Reporter:	Doktor Notor <notordoktor>
Component:	New packages	Assignee:	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...