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

Bug 955708

Summary: app-editors/zed: forces using LTO
Product: Gentoo Linux Reporter: Sam James <sam>
Component: Current packagesAssignee: Robert G. Siebeck <gentoo.2019>
Status: RESOLVED FIXED    
Severity: normal CC: freijon, proxy-maint
Priority: Normal Keywords: PullRequest
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://bugs.gentoo.org/show_bug.cgi?id=940493
https://bugs.gentoo.org/show_bug.cgi?id=949112
https://github.com/gentoo/gentoo/pull/42021
https://bugs.gentoo.org/show_bug.cgi?id=955788
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 955559    

Description Sam James archtester Gentoo Infrastructure gentoo-dev Security 2025-05-10 01:23:32 UTC
app-editors/zed requires a large amount of time, CPU, and RAM to build - in part because it unconditionally builds using LTO.

I don't think this is a reasonable thing for a package to force and by policy, we undo that in other packages. What we sometimes do is include a warning if LTO is not enabled if upstream recommend that it is.

But right now, it consumes a lot of arch testing resources for stabilisation and doesn't seem particularly worth it.
Comment 1 Larry the Git Cow gentoo-dev 2025-05-15 08:17:34 UTC
The bug has been closed via the following commit(s):

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

commit 3fcfda0716efcd72dcdbf0bae32512102917912e
Author:     Robert Siebeck <gentoo.2019@r123.de>
AuthorDate: 2025-05-14 20:17:02 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2025-05-15 08:17:30 +0000

    app-editors/zed: add 0.186.8
    
    Disable lto by default, enable it only when tc-is-lto returns true.
    
    Closes: https://bugs.gentoo.org/955708
    Signed-off-by: Robert Siebeck <gentoo.2019@r123.de>
    Part-of: https://github.com/gentoo/gentoo/pull/42021
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 app-editors/zed/Manifest           |   2 +
 app-editors/zed/zed-0.186.8.ebuild | 235 +++++++++++++++++++++++++++++++++++++
 2 files changed, 237 insertions(+)