Skip to content

Conversation

@hswong3i
Copy link
Contributor

Refer to nabijaczleweli/systemd-zram-generator.deb@8f2f3fa, building package on Debian 10+ and Ubuntu 20.04+ will fail with lto = true.

Moreover, tested with https://build.opensuse.org/package/show/home:alvistack:systemd/zram-generator, lto = "thin" working well for all support OS, including CentOS/Fedora/Debian/Ubuntu/openSUSE.

Signed-off-by: Wong Hoi Sing Edison [email protected]

Refer to <nabijaczleweli/systemd-zram-generator.deb@8f2f3fa>,
building package on Debian 10+ and Ubuntu 20.04+ will fail with `lto = true`.

Moreover, tested with
<https://build.opensuse.org/package/show/home:alvistack:systemd/zram-generator>,
`lto = "thin"` working well for all support OS, including
CentOS/Fedora/Debian/Ubuntu/openSUSE.

Signed-off-by: Wong Hoi Sing Edison <[email protected]>
@keszybz
Copy link
Member

keszybz commented Aug 8, 2021

Looking at https://doc.rust-lang.org/cargo/reference/profiles.html#lto, "thin" seems appropriate.

@keszybz
Copy link
Member

keszybz commented Aug 8, 2021

CI is failing when trying to load zram, i.e. lack of #92. Let's merge anyway.

@keszybz keszybz merged commit 17fc632 into systemd:main Aug 8, 2021
@hswong3i hswong3i deleted the main-cargo_toml branch August 9, 2021 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants