debcargo-conf/rust_hacks.md
2023-03-12 22:19:25 +01:00

9.5 KiB

Scope of this document

This document aims to document some nice "hacks" and tricks to employ when packaging crates.

Getting an overview of missing crates

A nice tool to generate a graphical overview of a rust projects' dependency tree is cargo debstatus. Install it like that: cargo install cargo-debstatus. Then download either a release or clone the git project and cd into there. Run cargo debstatus to get a nice graph about dependencies and reverse dependencies.

Patching crates

If a crate needs a) a newer dependency or b) an older dependency than the one in the archive you need to patch the crate. This is relatively common. You can also use this to patch out features in Cargo.toml or make changes to the source code.

Since the source is pulled from crates.io and not from github/lab, patching requires downloading the source from there. There are two ways to achieve this:

  1. Install cargo-download: cargo install cargo-download
    1.1) Download the source: cargo download foocrate version > foocrate.tar.gz
    1.2) Extract the crate: tar -xf foocrate.tar.gz
    1.3) cd foocrate-version && git init && git add . && git commit -m "d"
    1.4) Edit e.g. Cargo.toml to relax the dependencies: Instead of version 0.4 of a crate bump it to 0.5 ( if that is the corresponding debian version) or down to 0.3 (if that is the debian version)
    1.5) Generate a patch with your changes: git diff -p >> relax-deps.diff
    1.6) cp relax-deps.diff debcargo-conf/src/foocrate/debian
    1.7) cd debcargo-conf/src/foocrate/debian && mkdir patches
    1.8) mv relax-deps.diff patches/ && cd patches && echo relax-deps.diff >> series
    1.9) cd ../../../../
    1.10) run ./update.sh foocrate again. The patch should get applied and allow you to build against an older/newer dependency / with features disabled.
    1.11) Document your patches in d/changelog
  2. wget http://crates.io/api/v1/crates/foocrate/version/download -O foocrate-version.tar.gz
    Then follow 1.1)

Alternatively you can do it directly in the build directory with quilt:

  1. In the root directory of debcargo-conf, cd build/foo
  2. quilt series to check existing patches, quilt push -a to test apply all of them, and quilt pop -a to unapply them
  3. quilt new patch-name.patch to create a new patch, quilt edit path/to/file to edit a file with changes saved in current patch, quilt header -e --dep3 to add a DEP-3 patch header
  4. quilt refresh to update current patch and prevent fuzz (we don't allow fuzzes when building)
  5. cp -r debian/patches ../../src/foo/debian/ to copy updated patches over
  6. In root directory, run ./update.sh foo to update the build directory

Note that update.sh deletes and re-creates build/foo, so an open terminal in it needs to go up and down into the new directory.

capitol did a nice writeup which can be read here: https://blog.hackeriet.no/packaging-rust-part-II/

built-using-dh-cargo

If you get an error like this:

You must patch build.rs of CRATE to output 'println!(\"dh-cargo:deb-built-using=$lib=\$s={}\", env::var(\"CARGO_MANIFEST_DIR\").unwrap());' 
where: $s is 1 if the license(s) of the included static libs require source distribution alongside binaries, otherwise 0"

when building a FFi rust library you need to patch build.rs like stated above. $s is 0 for BSD-like licenses such as MIT and 1 for copyleft licenses like GPL.

Skipping tests / special d/rules overrides

Sometimes you get broken tests that can't be excluded or patched away. For instance if a test would require direct access to the RAM which isn't possible with sbuild. Then it is reasonable to skip those in d/rules.

  1. cd src/foocrate/debian && touch rules
  2. cd ../../../ && ./update.sh foocrate. This will generate a rules.debcargo.hint file you can use as template, similar to d/copyright
  3. cd src/foocrate/debian && cp rules.debcargo.hint rules
  4. use your favorite editor to edit the rules file to your needs. For skipping tests you can do the following:
override_dh_auto_test:
	dh_auto_test -- test -- --skip tests::broken_test --skip tests::second_broken_test

if those are the broken tests. Then cd in the top-level directory and run ./update.sh again. You might also need to skip the same tests in the autopkgtest runner. To do that, create a tests/control file in the debian/ folder.

Packaging binary crates

TODO

debcargo.toml tweaks

Excluding files

In case you need to exclude certain files from debcargo.toml, there is an easy way to do that. Just add excludes = ["foo/bar.rs", ""bar/non-dfsg-file"] in debcargo.toml. This has the following usecases:

  • Exclude non-dfsg/unnecessary files form the orig tarball (Example: svg-metadata) This is also required for some sys-crates to exclude vendored copies of the C library already in debian
  • Exclude broken tests that do not run (when in doubt, ask on #debian-rust) (Example: cxx )

Passing external packages to the buildsystem

If the crate needs external packages (such as -dev libraries) you can also pass those conveniently via debcargo.toml :
depends = ["libfoo-dev"] . This is essentially needed for all -sys crates which provide rust bindings to C developement libraries such as GTK, for instance.

Passing runtime test dependencies to autopkgtest

In rare cases the autopkgtest can fail on the official runners compared to the local one because it has a slightly different setup. You can do this like that: test_depends = ["foo"]. That happens very rarely, see cxx for an example.

Whitelisting files

Sometimes debcargo marks files as suspicious, most of the time those are tests written in C for -sys crates. Whitelist them like that: whitelist = [tests/foo.c"] . (Example: lidadwaita-sys)

Allowing alpha/beta dependencies

Some crates depend on a crate with an alpha/beta version strings. debcargo will emit an error if that is the case. To allow those deps, pass the following: allow_prerelease_deps = true. Do this only if you are sure this will work !

Collapsing features

If a crate has features, collapse_features = true should be set in debcargo.toml. This is strongly recommended. See issue #17 in the debcargo repo for the reasoning.

Marking feature tests as broken

Let's assume you are building a package and get the following output at the end:


autopkgtest [22:45:42]: test librust-foo-dev:: -----------------------]
autopkgtest [22:45:42]: test librust-foo-dev::  - - - - - - - - - - results - - - - - - - - - -
librust-foo-dev: PASS
autopkgtest [22:45:42]: @@@@@@@@@@@@@@@@@@@@ summary
rust-foo:@ FAIL non-zero exit status 101
librust-foo-dev:default PASS
librust-foo-dev: PASS
librust-foo-dev:serde: FAIL non-zero exit status 101

As we can see, the serde and @ feature fail. Since the other features (and the package itself) passes, we need to mark those test as flaky so the package can migrate to testing. Do it like that:

#serde and @ feature fail the autopkgtest
[packages.lib]
test_is_broken = false
[packages."lib+@"]
test_is_broken = true
[packages."lib+serde"]
test_is_broken = true

Examples: cxx, hashbrown, uom, ...

Do this only if some features fail. If all feature tests fail, read the test logs and look at the upstream test system. Make sure that some tests pass or that the tests aren't meant to be run. Some upstream projects run all test in a specific container or use specific setup. Also some crates are only tested with their default features enabled by upstream. If that's the case mark all test as broken:

# tests need a postgres container to run
[packages.lib]
test_is_broken = true

Examples: tokio-postgres, ...

Sometime you need to combine some tricks to run at least some tests: Exclude a faulty test, patch dependencies away and mark tests as broken.

More resources

For a full documentation of all keywords available in debcargo.toml refer to debcargo.toml.example in the debcargo repo.

Arch-specific failures

It can rarely happen that tests (read: autopkgtest) fail on specific arches because how bytes are addressed on that arch. The best course of action is to investigate first if it indeed is an arch-specific failure. If that's the case you need to write a patch that skips those faulty tests (on that arch) so the package can enter testing. Because arch names are different in rust, here is a handy table comparing them:

Debian arch name rust arch name (target_arch)
Arches autopkgtest runs on (needed for testing migration)
amd64 x86-64
i386 x86
arm64 aarch64
armel arm
armhf arm
ppc64el powerpc64
s390x powerpc64?
Other official arches¹
mipsel mips?
mips64el mips64
Unoffical ports with rustc/cargo (not really relevant)
powerpc powerpc?
ppc64 powerpc?
riscv64 riscv64
sparc64 sparc64?
x32 ?

Arches without rustc/cargo:

  • sh4²
  • alpha
  • arc
  • hppa
  • hurd-i386
  • ia64
  • kfreebsd-amd64
  • kfreebsd-i386
  • m68k

Only the first seven are really relevant, I included the rest for completeness's sake. If you encounter a test failure e.g. on armel, add this macro before the #[test] macro:
#[cfg(not(target_arch = "arm"))].
Then generate a patch with the changes and include it in the usual way. Also notify upstream that this arch is broken and send them your patch.
Footnotes: [1] https://wiki.debian.org/SupportedArchitectures [2] sh4 has only one test failure for cargo