Skip to content

Commit cd1d68b

Browse files
committed
ci: Forward RUSTFLAGS to docker via EXTRA_RUSTFLAGS
This needs to be a separate environment variable since otherwise it overwrites `RUSTFLAGS` as set in CI. This is currently happening, which is why warnings were not getting denied.
1 parent fb1901d commit cd1d68b

File tree

11 files changed

+21
-16
lines changed

11 files changed

+21
-16
lines changed

ci/docker/aarch64-unknown-linux-musl/Dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,6 @@ RUN /install-musl.sh aarch64 "$MUSL_VERSION"
2020
# FIXME: shouldn't need the `-lgcc` here, shouldn't that be in std?
2121
ENV PATH=$PATH:/musl-aarch64/bin:/rust/bin \
2222
CC_aarch64_unknown_linux_musl=musl-gcc \
23-
RUSTFLAGS='-Clink-args=-lgcc -L /musl-aarch64/lib' \
2423
CARGO_TARGET_AARCH64_UNKNOWN_LINUX_MUSL_LINKER=musl-gcc \
25-
CARGO_TARGET_AARCH64_UNKNOWN_LINUX_MUSL_RUNNER="qemu-aarch64 -L /musl-aarch64"
24+
CARGO_TARGET_AARCH64_UNKNOWN_LINUX_MUSL_RUNNER="qemu-aarch64 -L /musl-aarch64" \
25+
EXTRA_RUSTFLAGS="-Clink-args=-lgcc -L /musl-aarch64/lib"

ci/docker/arm-unknown-linux-musleabihf/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,4 +24,4 @@ ENV PATH=$PATH:/musl-arm/bin:/rust/bin \
2424
CC_arm_unknown_linux_musleabihf=musl-gcc \
2525
CARGO_TARGET_ARM_UNKNOWN_LINUX_MUSLEABIHF_LINKER=musl-gcc \
2626
CARGO_TARGET_ARM_UNKNOWN_LINUX_MUSLEABIHF_RUNNER="qemu-arm -L /musl-arm" \
27-
RUSTFLAGS="-L /musl-arm/lib"
27+
EXTRA_RUSTFLAGS="-L /musl-arm/lib"

ci/docker/i686-unknown-linux-musl/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,4 +22,4 @@ RUN /install-musl.sh i686 "$MUSL_VERSION"
2222

2323
ENV PATH=$PATH:/musl-i686/bin:/rust/bin \
2424
CC_i686_unknown_linux_musl=musl-gcc \
25-
RUSTFLAGS="-L /musl-i686/lib"
25+
EXTRA_RUSTFLAGS="-L /musl-i686/lib"

ci/docker/loongarch64-unknown-linux-musl/Dockerfile

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,10 @@ ARG MUSL_VERSION
1717
COPY install-musl.sh /
1818
RUN /install-musl.sh loongarch64 "$MUSL_VERSION"
1919

20-
ENV CARGO_TARGET_LOONGARCH64_UNKNOWN_LINUX_MUSL_LINKER=musl-gcc \
21-
CARGO_TARGET_LOONGARCH64_UNKNOWN_LINUX_MUSL_RUNNER="qemu-loongarch64" \
22-
CC_loongarch64_unknown_linux_musl=musl-gcc \
20+
ENV CC_loongarch64_unknown_linux_musl=musl-gcc \
2321
CFLAGS_loongarch64_unknown_linux_musl="-mabi=lp64d -fPIC" \
24-
RUSTFLAGS="-Ctarget-feature=+crt-static" \
22+
CARGO_TARGET_LOONGARCH64_UNKNOWN_LINUX_MUSL_LINKER=musl-gcc \
23+
CARGO_TARGET_LOONGARCH64_UNKNOWN_LINUX_MUSL_RUNNER="qemu-loongarch64" \
24+
EXTRA_RUSTFLAGS="-Ctarget-feature=+crt-static" \
2525
QEMU_LD_PREFIX=/musl-loongarch64 \
2626
PATH=$PATH:/musl-loongarch64/bin:/rust/bin

ci/docker/powerpc64le-unknown-linux-musl/Dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,6 @@ RUN /install-musl.sh powerpc64le "$MUSL_VERSION"
2020
# FIXME: shouldn't need the `-lgcc` here, shouldn't that be in std?
2121
ENV PATH=$PATH:/musl-powerpc64/bin:/rust/bin \
2222
CC_powerpc64le_unknown_linux_musl=musl-gcc \
23-
RUSTFLAGS='-Clink-args=-lgcc -L /musl-powerpc64/lib' \
2423
CARGO_TARGET_POWERPC64LE_UNKNOWN_LINUX_MUSL_LINKER=musl-gcc \
25-
CARGO_TARGET_POWERPC64LE_UNKNOWN_LINUX_MUSL_RUNNER="qemu-ppc64le -L /musl-powerpc64"
24+
CARGO_TARGET_POWERPC64LE_UNKNOWN_LINUX_MUSL_RUNNER="qemu-ppc64le -L /musl-powerpc64" \
25+
EXTRA_RUSTFLAGS="-Clink-args=-lgcc -L /musl-powerpc64/lib"

ci/docker/s390x-unknown-linux-musl/Dockerfile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ COPY install-musl.sh /
1515
RUN /install-musl.sh s390x "$MUSL_VERSION"
1616

1717
# FIXME: shouldn't need the `-lgcc` here, shouldn't that be in std?
18-
ENV CARGO_TARGET_S390X_UNKNOWN_LINUX_GNU_LINKER=s390x-linux-gnu-gcc \
18+
ENV CC_s390x_unknown_linux_gnu=musl-gcc \
19+
CARGO_TARGET_S390X_UNKNOWN_LINUX_GNU_LINKER=s390x-linux-gnu-gcc \
1920
CARGO_TARGET_S390X_UNKNOWN_LINUX_GNU_RUNNER="qemu-s390x -L /musl-s390x" \
20-
CC_s390x_unknown_linux_gnu=musl-gcc \
21-
RUSTFLAGS='-Clink-args=-lgcc -L /musl-s390x/lib' \
21+
EXTRA_RUSTFLAGS="-Clink-args=-lgcc -L /musl-s390x/lib" \
2222
PATH=$PATH:/musl-s390x/bin:/rust/bin

ci/docker/wasm32-wasip1/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ RUN /wasi.sh
1010
# library.
1111
ENV CARGO_TARGET_WASM32_WASIP1_RUNNER=wasmtime \
1212
CARGO_TARGET_WASM32_WASIP1_LINKER=/opt/wasi-sdk/bin/clang \
13-
CARGO_TARGET_WASM32_WASIP1_RUSTFLAGS="-Clink-self-contained=n -lwasi-emulated-process-clocks" \
13+
EXTRA_RUSTFLAGS="-Clink-self-contained=n -lwasi-emulated-process-clocks" \
1414
CC_wasm32_wasip1=/opt/wasi-sdk/bin/clang \
1515
CFLAGS_wasm32_wasip1=-D_WASI_EMULATED_PROCESS_CLOCKS \
1616
PATH=$PATH:/rust/bin:/wasmtime

ci/docker/wasm32-wasip2/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ RUN /wasi.sh
99
# itself, this should be fixed upstream.
1010
ENV CARGO_TARGET_WASM32_WASIP2_RUNNER=wasmtime \
1111
CARGO_TARGET_WASM32_WASIP2_LINKER=/opt/wasi-sdk/bin/clang \
12-
CARGO_TARGET_WASM32_WASIP2_RUSTFLAGS="-Clink-self-contained=n -lwasi-emulated-process-clocks -Clink-arg=-Wl,--export,cabi_realloc" \
12+
EXTRA_RUSTFLAGS="-Clink-self-contained=n -lwasi-emulated-process-clocks -Clink-arg=-Wl,--export,cabi_realloc" \
1313
CC_wasm32_wasip2=/opt/wasi-sdk/bin/clang \
1414
CFLAGS_wasm32_wasip2=-D_WASI_EMULATED_PROCESS_CLOCKS \
1515
PATH=$PATH:/rust/bin:/wasmtime

ci/docker/x86_64-unknown-linux-musl/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,4 +16,4 @@ COPY install-musl.sh /
1616
RUN /install-musl.sh x86_64 "$MUSL_VERSION"
1717

1818
ENV PATH=$PATH:/musl-x86_64/bin:/rust/bin \
19-
RUSTFLAGS="-L /musl-x86_64/lib"
19+
EXTRA_RUSTFLAGS="-L /musl-x86_64/lib"

ci/run-docker.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ run() {
5959
--user "$(id -u)":"$(id -g)" \
6060
--env LIBC_CI \
6161
--env LIBC_CI_ZBUILD_STD \
62+
--env RUSTFLAGS \
6263
--env RUSTDOCFLAGS \
6364
--env RUST_BACKTRACE \
6465
--env RUST_LIBC_UNSTABLE_GNU_FILE_OFFSET_BITS \

0 commit comments

Comments
 (0)