Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 739290

Summary: games-fps/xonotic-0.8.2 does not respect CFLAGS
Product: Gentoo Linux Reporter: Agostino Sarubbo <ago>
Component: Current packagesAssignee: Gentoo Games <games>
Status: RESOLVED WORKSFORME    
Severity: normal CC: alexey+gentoo, ionen
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 59506    
Attachments: build.log

Description Agostino Sarubbo gentoo-dev 2020-08-27 16:16:05 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: games-fps/xonotic-0.8.2 does not respect CFLAGS.
Discovered on: amd64 (internal ref: tinderbox)

NOTE:
To reproduce this issue you may want to add "-frecord-gcc-switches" to CFLAGS/CXXFLAGS/FFLAGS/FCFLAGS. If this is not something about c/c++ context see this bug as an hint to hide the QA warning (with QA_FLAGS_IGNORED) where is not possible to respect CFLAGS.
Comment 1 Agostino Sarubbo gentoo-dev 2020-08-27 16:16:08 UTC
Created attachment 657146 [details]
build.log

build log and emerge --info
Comment 2 Alexey 2021-08-31 23:29:24 UTC
I can't reproduce, it passes CFLAGS fine, even with "-frecord-gcc-switches" in all 4 flags.
Comment 3 Agostino Sarubbo gentoo-dev 2021-09-01 07:10:57 UTC
(In reply to Alexey from comment #2)
> I can't reproduce, it passes CFLAGS fine, even with "-frecord-gcc-switches"
> in all 4 flags.

if you think the check if wrong, please get in touch with portage devs.
Comment 4 Ionen Wolkens gentoo-dev 2021-09-01 08:21:05 UTC
Are you still getting the QA warning?

Thing is that I can't seem to get it either (including with clang), and I know these usually work for me. Not that I tried every possible options.
Comment 5 Ionen Wolkens gentoo-dev 2021-09-01 08:23:21 UTC
(In reply to Ionen Wolkens from comment #4)
> Are you still getting the QA warning?
> 
> Thing is that I can't seem to get it either (including with clang), and I
> know these usually work for me. Not that I tried every possible options.
...my impression is that this may have been an issue with the toolchain a year ago, I mentioned clang because it's what the build.log is using
Comment 6 Alexey 2021-09-01 08:28:16 UTC
I've just reproduced the QA warning with a dummy ebuild I wrote which definitely ignores CFLAGS, so the warning itself works for me.

I agree that it probably was some issue with the toolchain.