Skip to content

Commit 3018dbf

Browse files
committed
Auto merge of #153372 - homersimpsons:chore/fix-LegacyKeyValueFormat-x86_64-gnu, r=<try>
Fix LegacyKeyValueFormat report from docker build: x86_64-gnu try-job: x86_64-gnu-aux try-job: x86_64-gnu-debug try-job: x86_64-gnu-distcheck try-job: x86_64-gnu-gcc try-job: x86_64-gnu-llvm-21 try-job: x86_64-gnu-miri try-job: x86_64-gnu-nopt try-job: x86_64-gnu-tools try-job: x86_64-gnu
2 parents fe7294f + d19a07b commit 3018dbf

File tree

9 files changed

+37
-44
lines changed

9 files changed

+37
-44
lines changed

‎src/ci/docker/host-x86_64/x86_64-gnu-aux/Dockerfile‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,5 +29,5 @@ RUN sh /scripts/sccache.sh
2929
ENV NO_DEBUG_ASSERTIONS=1
3030
ENV NO_OVERFLOW_CHECKS=1
3131

32-
ENV RUST_CONFIGURE_ARGS --build=x86_64-unknown-linux-gnu
33-
ENV RUST_CHECK_TARGET check-aux
32+
ENV RUST_CONFIGURE_ARGS="--build=x86_64-unknown-linux-gnu"
33+
ENV RUST_CHECK_TARGET="check-aux"

‎src/ci/docker/host-x86_64/x86_64-gnu-debug/Dockerfile‎

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -29,20 +29,19 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
2929
COPY scripts/sccache.sh /scripts/
3030
RUN sh /scripts/sccache.sh
3131

32-
ENV RUSTBUILD_FORCE_CLANG_BASED_TESTS 1
32+
ENV RUSTBUILD_FORCE_CLANG_BASED_TESTS="1"
3333

3434
# llvm.use-linker conflicts with downloading CI LLVM
35-
ENV NO_DOWNLOAD_CI_LLVM 1
35+
ENV NO_DOWNLOAD_CI_LLVM="1"
3636

37-
ENV RUST_CONFIGURE_ARGS \
38-
--build=x86_64-unknown-linux-gnu \
37+
ENV RUST_CONFIGURE_ARGS="--build=x86_64-unknown-linux-gnu \
3938
--enable-debug \
4039
--enable-lld \
4140
--set rust.debuginfo-level-tests=2 \
4241
--set llvm.use-linker=lld \
4342
--set target.x86_64-unknown-linux-gnu.linker=clang \
4443
--set target.x86_64-unknown-linux-gnu.cc=clang \
45-
--set target.x86_64-unknown-linux-gnu.cxx=clang++
44+
--set target.x86_64-unknown-linux-gnu.cxx=clang++"
4645

4746
# This job checks:
4847
# - That ui tests can be built with `-Cdebuginfo=1`
@@ -53,7 +52,6 @@ ENV RUST_CONFIGURE_ARGS \
5352
# - That the tests with `//@ needs-force-clang-based-tests` pass, since they
5453
# don't run by default unless RUSTBUILD_FORCE_CLANG_BASED_TESTS is set.
5554

56-
ENV SCRIPT \
57-
python3 ../x.py --stage 2 build && \
55+
ENV SCRIPT="python3 ../x.py --stage 2 build && \
5856
python3 ../x.py --stage 2 test tests/ui && \
59-
python3 ../x.py --stage 2 test tests/run-make tests/run-make-cargo
57+
python3 ../x.py --stage 2 test tests/run-make tests/run-make-cargo"

‎src/ci/docker/host-x86_64/x86_64-gnu-distcheck/Dockerfile‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# Runs `distcheck`, which is a collection of smoke tests:
2-
#
2+
#
33
# - Run `make check` from an unpacked dist tarball to make sure we can at the
44
# minimum run check steps from those sources.
55
# - Check that selected dist components at least have expected directory shape
@@ -34,6 +34,6 @@ COPY scripts/sccache.sh /scripts/
3434
RUN sh /scripts/sccache.sh
3535

3636
# Make distcheck builds faster
37-
ENV DISTCHECK_CONFIGURE_ARGS "--enable-sccache"
37+
ENV DISTCHECK_CONFIGURE_ARGS="--enable-sccache"
3838

39-
ENV SCRIPT python3 ../x.py test distcheck
39+
ENV SCRIPT="python3 ../x.py test distcheck"

‎src/ci/docker/host-x86_64/x86_64-gnu-gcc/Dockerfile‎

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,15 +31,14 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
3131
COPY scripts/sccache.sh /scripts/
3232
RUN sh /scripts/sccache.sh
3333

34-
ENV NO_DEBUG_ASSERTIONS 1
35-
ENV RUST_CONFIGURE_ARGS \
36-
--build=x86_64-unknown-linux-gnu \
34+
ENV NO_DEBUG_ASSERTIONS="1"
35+
ENV RUST_CONFIGURE_ARGS="--build=x86_64-unknown-linux-gnu \
3736
--enable-sanitizers \
3837
--enable-profiler \
3938
--enable-compiler-docs \
4039
--set llvm.libzstd=true \
41-
--set 'rust.codegen-backends=[\"llvm\",\"gcc\"]'
42-
ENV SCRIPT python3 ../x.py \
40+
--set rust.codegen-backends=[\\\"llvm\\\",\\\"gcc\\\"]"
41+
ENV SCRIPT="python3 ../x.py \
4342
--stage 2 \
4443
test tests \
4544
--test-codegen-backend gcc \
@@ -51,4 +50,4 @@ ENV SCRIPT python3 ../x.py \
5150
--skip tests/rustdoc-js-std \
5251
--skip tests/rustdoc-json \
5352
--skip tests/rustdoc-ui \
54-
--set 'rust.codegen-backends=[\"llvm\",\"gcc\"]'
53+
--set rust.codegen-backends=[\\\"llvm\\\",\\\"gcc\\\"]"

‎src/ci/docker/host-x86_64/x86_64-gnu-llvm-21/Dockerfile‎

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -44,16 +44,15 @@ RUN sh /scripts/sccache.sh
4444

4545
# We are disabling CI LLVM since this builder is intentionally using a host
4646
# LLVM, rather than the typical src/llvm-project LLVM.
47-
ENV NO_DOWNLOAD_CI_LLVM 1
48-
ENV EXTERNAL_LLVM 1
47+
ENV NO_DOWNLOAD_CI_LLVM="1"
48+
ENV EXTERNAL_LLVM="1"
4949

5050
# Using llvm-link-shared due to libffi issues -- see #34486
51-
ENV RUST_CONFIGURE_ARGS \
52-
--build=x86_64-unknown-linux-gnu \
51+
ENV RUST_CONFIGURE_ARGS="--build=x86_64-unknown-linux-gnu \
5352
--llvm-root=/usr/lib/llvm-21 \
5453
--enable-llvm-link-shared \
5554
--set rust.randomize-layout=true \
56-
--set rust.thin-lto-import-instr-limit=10
55+
--set rust.thin-lto-import-instr-limit=10"
5756

5857
COPY scripts/shared.sh /scripts/
5958

@@ -63,4 +62,4 @@ COPY scripts/x86_64-gnu-llvm3.sh /scripts/
6362
COPY scripts/stage_2_test_set1.sh /scripts/
6463
COPY scripts/stage_2_test_set2.sh /scripts/
6564

66-
ENV SCRIPT "Must specify DOCKER_SCRIPT for this image"
65+
ENV SCRIPT="Must specify DOCKER_SCRIPT for this image"

‎src/ci/docker/host-x86_64/x86_64-gnu-miri/Dockerfile‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,16 +34,15 @@ ENV GCC_EXEC_PREFIX="/usr/lib/gcc/"
3434

3535
COPY host-x86_64/x86_64-gnu-miri/check-miri.sh /tmp/
3636

37-
ENV RUST_CONFIGURE_ARGS \
38-
--build=x86_64-unknown-linux-gnu \
39-
--enable-new-symbol-mangling
37+
ENV RUST_CONFIGURE_ARGS="--build=x86_64-unknown-linux-gnu \
38+
--enable-new-symbol-mangling"
4039

41-
ENV HOST_TARGET x86_64-unknown-linux-gnu
40+
ENV HOST_TARGET="x86_64-unknown-linux-gnu"
4241

4342
# FIXME(#133381): currently rustc alt builds do *not* have rustc debug
4443
# assertions enabled! Therefore, we cannot force download CI rustc.
4544
#ENV FORCE_CI_RUSTC 1
4645

4746
COPY scripts/shared.sh /scripts/
4847

49-
ENV SCRIPT /tmp/check-miri.sh ../x.py
48+
ENV SCRIPT="/tmp/check-miri.sh ../x.py"

‎src/ci/docker/host-x86_64/x86_64-gnu-nopt/Dockerfile‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
2222
COPY scripts/sccache.sh /scripts/
2323
RUN sh /scripts/sccache.sh
2424

25-
ENV RUST_CONFIGURE_ARGS --build=x86_64-unknown-linux-gnu \
25+
ENV RUST_CONFIGURE_ARGS="--build=x86_64-unknown-linux-gnu \
2626
--disable-optimize-tests \
27-
--set rust.test-compare-mode
28-
ENV SCRIPT python3 ../x.py test --stage 1 --set rust.optimize=false library/std \
29-
&& python3 ../x.py --stage 2 test
27+
--set rust.test-compare-mode"
28+
ENV SCRIPT="python3 ../x.py test --stage 1 --set rust.optimize=false library/std \
29+
&& python3 ../x.py --stage 2 test"

‎src/ci/docker/host-x86_64/x86_64-gnu-tools/Dockerfile‎

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -76,18 +76,17 @@ COPY scripts/nodejs.sh /scripts/
7676
RUN sh /scripts/nodejs.sh /node
7777
ENV PATH="/node/bin:${PATH}"
7878

79-
ENV RUST_CONFIGURE_ARGS \
80-
--build=x86_64-unknown-linux-gnu \
79+
ENV RUST_CONFIGURE_ARGS="--build=x86_64-unknown-linux-gnu \
8180
--save-toolstates=/tmp/toolstate/toolstates.json \
82-
--enable-new-symbol-mangling
81+
--enable-new-symbol-mangling"
8382

84-
ENV HOST_TARGET x86_64-unknown-linux-gnu
83+
ENV HOST_TARGET="x86_64-unknown-linux-gnu"
8584

8685
# FIXME(#133381): currently rustc alt builds do *not* have rustc debug
8786
# assertions enabled! Therefore, we cannot force download CI rustc.
8887
#ENV FORCE_CI_RUSTC 1
8988

9089
COPY scripts/shared.sh /scripts/
9190

92-
ENV SCRIPT /tmp/checktools.sh ../x.py && \
93-
python3 ../x.py test tests/rustdoc-gui --stage 2 --test-args "'--jobs 1'"
91+
ENV SCRIPT="/tmp/checktools.sh ../x.py && \
92+
python3 ../x.py test tests/rustdoc-gui --stage 2 --test-args '--jobs 1'"

‎src/ci/docker/host-x86_64/x86_64-gnu/Dockerfile‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,9 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
2424
COPY scripts/sccache.sh /scripts/
2525
RUN sh /scripts/sccache.sh
2626

27-
ENV RUST_CONFIGURE_ARGS \
28-
--build=x86_64-unknown-linux-gnu \
27+
ENV RUST_CONFIGURE_ARGS="--build=x86_64-unknown-linux-gnu \
2928
--enable-sanitizers \
3029
--enable-profiler \
3130
--enable-compiler-docs \
32-
--set llvm.libzstd=true
33-
ENV SCRIPT python3 ../x.py --stage 2 test
31+
--set llvm.libzstd=true"
32+
ENV SCRIPT="python3 ../x.py --stage 2 test"

0 commit comments

Comments
 (0)