Skip to content

Commit f9097f8

Browse files
committed
Auto merge of #112633 - Kobzol:ci-msvc-merge, r=pietroalbini
CI: merge `msvc` test CI jobs Merges `msvc` jobs together to save CI time. Currently, both runners take about 1h 15 minutes, but nowadays it should be possible to just run everything in a single job. CI run: https://github.com/rust-lang/rust/actions/runs/5272144087/jobs/9534015536?pr=112633 (both finish under ~1h 35 minutes) After this change, we no longer test both `x.py` and `x.ps1`, but I don't suppose that it's worth it to spend 1.5 hours of additional CI time just for that. I suggest to run all tests using e.g. `x.py` and then run just `x.ps1 test --stage 2 --force-rerun tests/<single-quick-test>`. Also I'm not sure if it's worth it to keep using the Makefile for this.
2 parents 4996b56 + 895eb30 commit f9097f8

File tree

3 files changed

+11
-39
lines changed

3 files changed

+11
-39
lines changed

.github/workflows/ci.yml

+4-14
Original file line numberDiff line numberDiff line change
@@ -383,25 +383,15 @@ jobs:
383383
DIST_REQUIRE_ALL_TOOLS: 1
384384
JEMALLOC_SYS_WITH_LG_PAGE: 14
385385
os: macos-latest
386-
- name: x86_64-msvc-1
386+
- name: x86_64-msvc
387387
env:
388388
RUST_CONFIGURE_ARGS: "--build=x86_64-pc-windows-msvc --enable-profiler"
389-
SCRIPT: make ci-subset-1
389+
SCRIPT: make ci-msvc
390390
os: windows-2019-8core-32gb
391-
- name: x86_64-msvc-2
392-
env:
393-
RUST_CONFIGURE_ARGS: "--build=x86_64-pc-windows-msvc --enable-profiler"
394-
SCRIPT: make ci-subset-2
395-
os: windows-2019-8core-32gb
396-
- name: i686-msvc-1
397-
env:
398-
RUST_CONFIGURE_ARGS: "--build=i686-pc-windows-msvc"
399-
SCRIPT: make ci-subset-1
400-
os: windows-2019-8core-32gb
401-
- name: i686-msvc-2
391+
- name: i686-msvc
402392
env:
403393
RUST_CONFIGURE_ARGS: "--build=i686-pc-windows-msvc"
404-
SCRIPT: make ci-subset-2
394+
SCRIPT: make ci-msvc
405395
os: windows-2019-8core-32gb
406396
- name: x86_64-msvc-cargo
407397
env:

src/bootstrap/mk/Makefile.in

+3-9
Original file line numberDiff line numberDiff line change
@@ -57,17 +57,11 @@ tidy:
5757
prepare:
5858
$(Q)$(BOOTSTRAP) build --stage 2 nonexistent/path/to/trigger/cargo/metadata
5959

60-
TESTS_IN_2 := \
61-
tests/ui \
62-
src/tools/linkchecker
63-
6460
## MSVC native builders
6561

66-
# these intentionally don't use `$(BOOTSTRAP)` so we can test the shebang on Windows
67-
ci-subset-1:
68-
$(Q)$(CFG_SRC_DIR)/x.py test --stage 2 $(TESTS_IN_2:%=--exclude %)
69-
ci-subset-2:
70-
$(Q)$(CFG_SRC_DIR)/x.ps1 test --stage 2 $(TESTS_IN_2)
62+
# this intentionally doesn't use `$(BOOTSTRAP)` so we can test the shebang on Windows
63+
ci-msvc:
64+
$(Q)$(CFG_SRC_DIR)/x.py test --stage 2
7165

7266
## MingW native builders
7367

src/ci/github-actions/ci.yml

+4-16
Original file line numberDiff line numberDiff line change
@@ -582,28 +582,16 @@ jobs:
582582
# Windows Builders #
583583
######################
584584

585-
- name: x86_64-msvc-1
585+
- name: x86_64-msvc
586586
env:
587587
RUST_CONFIGURE_ARGS: --build=x86_64-pc-windows-msvc --enable-profiler
588-
SCRIPT: make ci-subset-1
588+
SCRIPT: make ci-msvc
589589
<<: *job-windows-8c
590590

591-
- name: x86_64-msvc-2
592-
env:
593-
RUST_CONFIGURE_ARGS: --build=x86_64-pc-windows-msvc --enable-profiler
594-
SCRIPT: make ci-subset-2
595-
<<: *job-windows-8c
596-
597-
- name: i686-msvc-1
598-
env:
599-
RUST_CONFIGURE_ARGS: --build=i686-pc-windows-msvc
600-
SCRIPT: make ci-subset-1
601-
<<: *job-windows-8c
602-
603-
- name: i686-msvc-2
591+
- name: i686-msvc
604592
env:
605593
RUST_CONFIGURE_ARGS: --build=i686-pc-windows-msvc
606-
SCRIPT: make ci-subset-2
594+
SCRIPT: make ci-msvc
607595
<<: *job-windows-8c
608596

609597
- name: x86_64-msvc-cargo

0 commit comments

Comments
 (0)