Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 120591 - app-i18n/skim uses CFLAGS for building C++ code
Summary: app-i18n/skim uses CFLAGS for building C++ code
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: CJK Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-01-27 17:29 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2006-02-16 11:02 UTC (History)
0 users

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

Attachments
Ebuild patch (skim-ebuild.patch,588 bytes, patch)
2006-01-27 17:29 UTC, Diego Elio Pettenò (RETIRED) 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Elio Pettenò (RETIRED) gentoo-dev 2006-01-27 17:29:34 UTC 
As the summary says, skim is using CFLAGS but it builds only C++ code (as it's KDE stuff anyway), the problem is that people might have C++ specific flags in CXXFLAGS or might be using something like -Wno-pointer-sign in CFLAGS (as I am O:) ), throwing lots of unneeded warnings when misusing the two.
Unfortunately the proper fix in scons is all but simple, I'm attaching a workaround in ebuild that solves the problem.
If someone has contact with upstream, maybe they can fix that in next releases, if it's not scons being stupid.

HTH,
Diego
Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-01-27 17:29:57 UTC 
Created attachment 78314 [details, diff]
Ebuild patch
Comment 2 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-02-02 06:17:04 UTC 
By the way, 1.4.4 is released.
Comment 3 Alastair Tse (RETIRED) gentoo-dev 2006-02-16 11:02:10 UTC 
fixing the unstable ebuilds to use this. thanks. also bumped 1.4.4