Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 955708 - app-editors/zed: forces using LTO
Summary: app-editors/zed: forces using LTO
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Robert G. Siebeck
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks: 955559
  Show dependency tree
 
Reported: 2025-05-10 01:23 UTC by Sam James
Modified: 2025-05-15 08:17 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 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(+)