Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 480658 - sys-devel/llvm flawed check-reqs logic
Summary: sys-devel/llvm flawed check-reqs logic
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Bernard Cafarelli
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-08-11 17:53 UTC by Julian Ospald
Modified: 2013-09-30 06:51 UTC (History)
2 users (show)

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 Julian Ospald 2013-08-11 17:53:05 UTC
elif is-flagq -g || is-flagq -ggdb; then

will give false-positives when I have e.g. "-g0" appended and die without purpose
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2013-08-11 20:07:22 UTC
Suggested fix?
Comment 2 Julian Ospald 2013-08-11 20:44:18 UTC
we'd probably need regexp support for is-flagq
Comment 3 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2013-08-11 20:49:43 UTC
Just to be clear, do you mean that the '-g' check matches '-g0', or you have something like '-g -g0'?
Comment 4 Julian Ospald 2013-08-11 20:50:33 UTC
I mean '-g -g0' or even '-g -some-other-flag -g0', yes
Comment 5 Julian Ospald 2013-08-11 20:51:13 UTC
I think that setup is not uncommon, because it's the easiest way to disable "-g" for a particular package without doing weird bash substitution or redefining the whole CFLAGS.
Comment 6 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2013-08-11 20:54:54 UTC
Supporting such a thing would probably require something like get-flag-pos or even more scary... and I told that introducing the check-reqs thing is going to be give more trouble than benefit, eh.

@toolchain, anything you could suggest?
Comment 7 SpanKY gentoo-dev 2013-09-30 00:51:23 UTC
(In reply to Michał Górny from comment #6)

we don't have any toolchain-funcs helpers atm that'd handle this

seems like it only comes up when people have -g in their own global make.conf and then additionally set -g0 in per-package env files.  i'd suspect this is uncommon.  to the point where the 5 people who do this and read ewarnings can suffer it ...
Comment 8 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2013-09-30 06:51:44 UTC
As vapier said, then.