Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

This is required for the right balance between the
smallest binary size and enough debug info to generate Apertis SBOM.

It's based on previous Apertis patches:
 - size-optimize.patch
 - Switch-release-builds-back-to-thin-LTO.patch

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Depending how apt parse the build-deps list, it can fail to
resolve dependencies with:

The following packages have unmet dependencies:
 libstd-rust-web-dev : Conflicts: libstd-rust-dev
 rustc-web : Conflicts: rustc but 1.63.0+dfsg1-2+apertis1bv2025dev3b1 is to be installed

Actually, doing a "apt build-dep ." works, but it fails after
a "wrap-and-sort" like it is used in ci-buildpackage. That is because
the order between dh-cargo and cargo:native is inverted. So, to avoid
potential conflicts, makes sure we don't pull the "-web" variants
which are only used to build firefox.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Ryan Gonzalez <ryan.gonzalez@collabora.com>

infrastructure/apertis-issues#431

Signed-off-by: Ryan Gonzalez <ryan.gonzalez@collabora.com>

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: Ryan Gonzalez <ryan.gonzalez@collabora.com>

infrastructure/apertis-issues#431

Signed-off-by: Ryan Gonzalez <ryan.gonzalez@collabora.com>

This is no longer needed without stdbuf present.

infrastructure/apertis-issues#431

Signed-off-by: Ryan Gonzalez <ryan.gonzalez@collabora.com>

The stdbuf build process involves building a shared library externally
and embedding it in the binary, which makes it difficult to extract
debug info from binaries and *then* still strip them. This is pretty
hard to fix while keeping coreutils cargo-installable, so just disable
stdbuf instead.

infrastructure/apertis-issues#431

Signed-off-by: Ryan Gonzalez <ryan.gonzalez@collabora.com>

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

In order to avoid confusions with the different types of package tests
that can be enabled rename the test metadata file to follow a better
pattern and rationale.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Rust binaries stripped at linking time are smaller than binaries
stripped with dh_strip. Since we want the smaller binary possible for
rust-coreutils, we strip debug symbols at linking time, but that
means dh_setup_copyright is then not able to extract required data.
As a workaround we build here a non-stripped binary only for
dh_setup_copyright.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Re-enable the running of bootstrap tests on MR which where disabled during
rebase.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Since .cargo/config already set some rust flags when setting them in
debian/rules these values get overwritten. In order to avoid this
issue and to keep consistency move the new flags to .cargo/config.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Detlev Casanova authored 1 year ago and Dylan Aïssi committed 1 year ago

- Add "lto": whole binary link time optimisation.
- Use opt-level = "z": to optimize for size.
- Make the panic handler simply abort() rather then unwinding and
   printing a backtrace.

Add flag in debian/rules to strip symbols from the binary.

Inspired by https://github.com/uutils/coreutils/commit/0c4f586



Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>
Signed-off-by: Sjoerd Simons <sjoerd@collabora.com>
Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Ritesh Raj Sarraf <ritesh.sarraf@collabora.co.uk>

Detlev Casanova authored 1 year ago and Dylan Aïssi committed 1 year ago

Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>

Detlev Casanova authored 1 year ago and Dylan Aïssi committed 1 year ago

This patch makes sure that tsort behaves the same way as in the C
version.

Signed-off-by: Detlev Casanova <detlev.casanova@collabora.com>

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Following the same approach as in cp and mv, add a hack to avoid failing
when scripts try to use -Z with mkdir.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Fix the support -Z in cp and mv which does not require any argument. This
fixes the implementation in 01d18805 which was buggy.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

In the new version of rust-coreutils additional checks for options are
perfomed, including the number of arguments. In order to properly support
the -Z option include the number of arguments it requires.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>