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

Bug 831423

Summary: sys-devel/gcc-12.2.1_p20221210 does not respect LDFLAGS
Product: Gentoo Linux Reporter: Agostino Sarubbo <ago>
Component: Current packagesAssignee: Gentoo Toolchain Maintainers <toolchain>
Status: RESOLVED FIXED    
Severity: normal CC: esigra
Priority: Normal Keywords: PullRequest
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://github.com/gentoo/gentoo/pull/33114
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 331933    
Attachments: build.log.xz

Description Agostino Sarubbo gentoo-dev 2022-01-18 19:50:14 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: sys-devel/gcc-11.2.1_p20220115 does not respect LDFLAGS.
Discovered on: amd64 (internal ref: ci)

NOTE:
To reproduce this issue you may want to set LDFLAGS="${LDFLAGS} -Wl,--defsym=__gentoo_check_ldflags__=0". If this is not something about c/c++ context you may want to see this bug as an hint to hide the QA warning (with QA_FLAGS_IGNORED) where is not possible to respect LDFLAGS.
Comment 1 Agostino Sarubbo gentoo-dev 2022-01-18 19:50:20 UTC
Created attachment 762520 [details]
build.log.xz

build log and emerge --info (compressed because it exceeds attachment limit, use 'xzless' to read it)
Comment 2 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-01-18 20:31:39 UTC
I was pretty sure we had a bug for this but I can't find it right now.
Comment 3 Alex Xu (Hello71) 2022-01-19 21:10:26 UTC
(In reply to Sam James from comment #2)
> I was pretty sure we had a bug for this but I can't find it right now.

at least five: https://bugs.gentoo.org/buglist.cgi?short_desc=gcc ldflags&short_desc_type=allwordssubstr
Comment 4 Agostino Sarubbo gentoo-dev 2022-06-26 08:43:27 UTC
lto_tinderbox has reproduced this issue with version 12.1.1_p20220625 - Updating summary.
Comment 5 Agostino Sarubbo gentoo-dev 2022-08-21 14:46:07 UTC
tinderbox_musl has reproduced this issue with version 12.2.0 - Updating summary.
Comment 6 Agostino Sarubbo gentoo-dev 2022-10-14 06:53:58 UTC
tinderbox_musl has reproduced this issue with version 12.2.1_p20221008 - Updating summary.
Comment 7 Agostino Sarubbo gentoo-dev 2022-12-05 07:47:30 UTC
ci has reproduced this issue with version 12.2.1_p20221203 - Updating summary.
Comment 8 Agostino Sarubbo gentoo-dev 2022-12-11 08:14:53 UTC
ci has reproduced this issue with version 12.2.1_p20221210 - Updating summary.
Comment 9 Larry the Git Cow gentoo-dev 2023-09-30 09:38:53 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=bb2d045c02a6ca647ef3280f4987cbc0d14e5a7e

commit bb2d045c02a6ca647ef3280f4987cbc0d14e5a7e
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2023-09-28 23:27:06 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2023-09-30 09:38:40 +0000

    toolchain.eclass: rework bootstrapping logic
    
    * Build stage1 compiler with user's CFLAGS. This consistently ends up
      saving at least 15 minutes for me on a fast amd64 machine and should save
      more on slower machines and architectures.
    
      There's only any risk here if the host compiler is ancient/very buggy and
      even then, you get a failed bootstrap later on. The GCC developers, per the
      linked bug, end up using STAGE1_CFLAGS="-O2" anyway to speed up the process
      so it's not like this is untested at all.
    
      mattst88 actually brought this up.. 10 years ago (bug #477548). Let's try
      make that right now.
    
    * Respect LDFLAGS for target libraries for native builds. Not touching this
      for cross builds, at least for now, as it's a bit more delicate.
    
      (Unfortunately, we have to put a hack in here for now until we can fix
      multilib.eclass - see bug #914881).
    
    Bug: https://gcc.gnu.org/PR111619
    Bug: https://bugs.gentoo.org/914881
    Closes: https://bugs.gentoo.org/477548
    Closes: https://bugs.gentoo.org/831423
    Closes: https://bugs.gentoo.org/840392
    Apologies-to: Matt Turner <mattst88@gentoo.org>
    Signed-off-by: Sam James <sam@gentoo.org>

 eclass/toolchain.eclass | 52 +++++++++++++++++++++++++++++++++----------------
 1 file changed, 35 insertions(+), 17 deletions(-)