Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 73830 - CFLAGS override in qstat ebuild causes -Dsysconfdir to not be defined in build
Summary: CFLAGS override in qstat ebuild causes -Dsysconfdir to not be defined in build
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Games (show other bugs)
Hardware: All Linux
: High normal
Assignee: Gentoo Games
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-12-08 12:19 UTC by Daniel Ceregatti
Modified: 2004-12-08 13:30 UTC (History)
0 users

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


Attachments
qstat-2.6.ebuild patch that defines sysconfdir=/etc (qstat.patch,324 bytes, patch)
2004-12-08 12:26 UTC, Daniel Ceregatti
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Ceregatti 2004-12-08 12:19:57 UTC
After emerging qstat-2.6 I try to query something that's defined in /etc/qstat.cfg, such as my customized armyops search, which I want available to all users, hence its presence in the global /etc/qstat.cfg:

gametype AA new extend GS2
        name = America's Army
        default port = 1717
end


This fails. Strace shows it never looks at /etc/qstat.cfg. I then re-emerge and observe the output and note that -Dsysconfdir isn't defined during the compile. I then check the ebuild, and sure enough, CFLAGS is completely overridden by the ebuild.

Typing "make" in the qstat source directory adds -Dsysconfdir=\"/etc\" to CFLAGS, thereby causing /etc/qstat.cfg to be read by qstat.

Reproducible: Always
Steps to Reproduce:
1. See details
2.
3.

Actual Results:  
qstat -aa fungus.sh.nu
unknown option "-aa"


Expected Results:  
qstat -aa fungus.sh.nu
ADDRESS           PLAYERS      MAP   RESPONSE TIME    NAME
fungus                 0/26 Insurgent Camp     18 / 0  AAGP_GameTeamObjective -=
Primer's Armyops - Linux =-
Comment 1 Daniel Ceregatti 2004-12-08 12:26:01 UTC
Created attachment 45555 [details, diff]
qstat-2.6.ebuild patch that defines sysconfdir=/etc

This works for me.
Comment 2 Mr. Bones. (RETIRED) gentoo-dev 2004-12-08 13:30:18 UTC
patch in cvs.  Thanks for the bug report.