Merge ~liushuyu-011/ubuntu/+source/rust-zram-generator:ubuntu/devel into ubuntu/+source/rust-zram-generator:ubuntu/devel

[Zixing Liu]

This fixes the debug information saving using the ThinLTO instead of the full LTO.
See https://github.com/rust-lang/rust/pull/89041 for the background information.

[Sebastien Bacher (seb128)]

Thank you for your contribution. It seems another change was made after that one was proposed which landed as https://launchpad.net/ubuntu/+source/rust-zram-generator/1.1.2-2ubuntu1. Is the current change still desirable? If so could you add a changelog entry describing the change and perhaps forward to Debian so we could keep the package synced?

review: Needs Fixing

[Zixing Liu (liushuyu-011)]

> https://launchpad.net/ubuntu/+source/rust-zram-generator/1.1.2-2ubuntu1

It looks like this one used a more brute-force way to fix the build (by disabling the debug information).

I think I will reject this merge proposal myself.

review: Disapprove

Unmerged commits

db065b1... by Zixing Liu on 2023-04-06

d/patches: use thinLTO to avoid LLVM generating bad debuginfo

diff --git a/debian/patches/series b/debian/patches/series
index 75abb7f..399b108 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1 +1,2 @@
 relax-dep.diff
+use-thin-lto.patch
diff --git a/debian/patches/use-thin-lto.patch b/debian/patches/use-thin-lto.patch
new file mode 100644
index 0000000..c8eefdb
--- /dev/null
+++ b/debian/patches/use-thin-lto.patch
@@ -0,0 +1,11 @@
+--- a/Cargo.toml	2023-04-04 16:28:38.868577487 -0600
++++ b/Cargo.toml	2023-04-04 16:28:47.176709209 -0600
+@@ -20,7 +20,7 @@
+ license = "MIT"
+ [profile.release]
+ opt-level = "z"
+-lto = true
++lto = "thin"
+ codegen-units = 1
+ panic = "abort"
+ [dependencies.anyhow]