Skip to content
Snippets Groups Projects
Commit c8a4582f authored by Ryan Gonzalez's avatar Ryan Gonzalez
Browse files

Switch release builds back to thin LTO


This was placed in a separate patch instead of merging into the existing
patch because that one is actually pulled from upstream, so we'd be
liable to accidentally drop the change when rust-coreutils is updated in
the future.

infrastructure/apertis-issues#431

Signed-off-by: default avatarRyan Gonzalez <ryan.gonzalez@collabora.com>
parent 49a2661c
No related branches found
No related tags found
3 merge requests!43Merge changes from apertis/v2024-updates into apertis/v2024,!40Backport v2024 <- v2025pre: Disable stdbuf,!39Disable stdbuf
From: Ryan Gonzalez <ryan.gonzalez@collabora.com>
Date: Tue, 17 Sep 2024 16:39:27 -0500
Subject: Switch release builds back to thin LTO
Using full LTO results in the loss of individual crate compilation unit
names in the debuginfo, which interferes with our copyright generation.
Rust's default behavior is *already* thin LTO, which only takes up 1MiB
of extra space but avoids this issue.
(This seems to work on newer Rust releases and thus either specific to
the Debian package or fixed in rustc/LLVM, so it might be worth re-
evaluating the situation once the version in Debian is updated.)
Signed-off-by: Ryan Gonzalez <ryan.gonzalez@collabora.com>
---
Cargo.toml | 1 -
1 file changed, 1 deletion(-)
diff --git a/Cargo.toml b/Cargo.toml
index 88a712a..c99ca67 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -428,6 +428,5 @@ path = "src/bin/uudoc.rs"
required-features = ["uudoc"]
[profile.release]
-lto = true
opt-level = "z"
panic = "abort"
...@@ -36,3 +36,4 @@ apertis/HACK-cp-mv-add-dummy-missing-arguments.patch ...@@ -36,3 +36,4 @@ apertis/HACK-cp-mv-add-dummy-missing-arguments.patch
apertis/Add-hack-for-missing-argument-to-mkdir.patch apertis/Add-hack-for-missing-argument-to-mkdir.patch
apertis/tsort-Switch-to-BTreeHash-and-BTreeSet.patch apertis/tsort-Switch-to-BTreeHash-and-BTreeSet.patch
apertis/size-optimize.patch apertis/size-optimize.patch
apertis/Switch-release-builds-back-to-thin-LTO.patch
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment