fix: remove dependcy on clang & mold
All checks were successful
Rust / build (push) Successful in 1m32s
ci/woodpecker/push/cron-docker-builder Pipeline was successful
ci/woodpecker/push/push-next Pipeline was successful
ci/woodpecker/push/tag-created Pipeline was successful
Release Please / Release-plz (push) Successful in 54s
ci/woodpecker/cron/cron-docker-builder Pipeline was successful
ci/woodpecker/cron/push-next Pipeline was successful
ci/woodpecker/cron/tag-created Pipeline was successful

This was only added to try and improve compile times.

Re-measuring the difference after months of work and refactoring, the
gain from the additional requirements was marginal (39.8s -> 37.5s).

So, to simplify the requirement, clang and mold have been removed.

Closes: kemitix/git-next#131
This commit is contained in:
Paul Campbell 2024-08-04 20:41:38 +01:00
parent 9a1756bf6c
commit 281c07c849
3 changed files with 1 additions and 17 deletions

View file

@ -1,11 +1,4 @@
# ./cargo/config.toml
[target.x86_64-unknown-linux-gnu]
linker = "/usr/bin/clang-16"
rustflags = [
"-C",
"link-arg=--ld-path=/usr/bin/mold",
]
[profile.dev]
debug = 0
strip = "debuginfo"

View file

@ -1,7 +1,7 @@
FROM docker.io/rust:1.80.0-bookworm
RUN apt-get update && \
apt-get install -y clang-16 mold libdbus-1-dev && \
apt-get install -y libdbus-1-dev && \
curl -L https://github.com/cargo-bins/cargo-binstall/releases/latest/download/cargo-binstall-x86_64-unknown-linux-musl.tgz -o cargo-binstall.tgz && \
tar -xzf cargo-binstall.tgz && \
rm cargo-binstall.tgz && \
@ -15,8 +15,6 @@ RUN cargo chef --version
RUN rustfmt --version
RUN cargo fmt --version
RUN cargo clippy --version
RUN mold --version
RUN clang-16 --version
RUN cargo --version
RUN rustc --version
RUN rustup --version

View file

@ -32,13 +32,6 @@ See [Behaviour](#behaviour) to learn how we do this.
- libdbus-1-dev (ubuntu/debian)
- dbus-devel (fedora)
### x86_64-unknown-linux-gnu
Additionally for this platform, to improved compilation times:
- clang-16
- mold
See `.cargo/config.toml` for how they are configured.
## Installation