Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 220393 - media-sound/kstreamripper-0.3.4 s/cflags/cxxflags/, MAKEOPTS issues
Summary: media-sound/kstreamripper-0.3.4 s/cflags/cxxflags/, MAKEOPTS issues
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] KDE (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Sound Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-05-05 15:54 UTC by Duncan
Modified: 2008-05-13 06:41 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Duncan 2008-05-05 15:54:52 UTC
U use some C++ unfriendly flags (-freorder-blocks-and-partition -combine) in CFLAGS that I of course don't have in CXXFLAGS.  I thus noticed that kstreamripper-0.3.4 seems to use CFLAGS to compile C++ code, generating unexpected warnings:

cc1plus: note: -freorder-blocks-and-partition does not work with exceptions

Suggested fix until upstream can change their sources appropriately, simply set CFLAGS="${CXXFLAGS}" in the ebuild so it uses the proper flag set.

Also, I'm not sure if -l<number> (load average job limits) is supposed to be supported in MAKEOPTS or not, but it's certainly very useful on most compiles.  Unfortunately kstreamripper doesn't like it, apparently due to the use of scons.  Setting a MAKEOPTS with just -j<number> in /etc/portage/env/media-sound/kstreamripper eliminates the problem for me, but it may be worth filtering -l from MAKEOPTS in the ebuild.  Actually, given the number of scons using packages, it may be worth an eclass function if there's not one already, perhaps in flagomatic?  (If you want this as a different bug, say so and I'll do it, but since it might be NOTABUG/WONTFIX... .)

(I don't believe emerge --info is necessary for this one.  Let me know if it is or if other info would be helpful.)
Duncan
Comment 1 Samuli Suominen (RETIRED) gentoo-dev 2008-05-06 18:02:10 UTC
Scons itself needs to be fixed as it doesn't handle the ${MAKEOPTS} correctly; best we can do here is to remove the ${MAKEOPTS} from ebuild. AFAIK.

But you are right, mixing CFLAGS with CXXFLAGS is plain wrong.
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2008-05-13 06:41:59 UTC
should be fixed now, reopen if not