Skip to content

Commit 8a4a6f7

Browse files
committed
msvc dist extended: clippy is optional
The windows tarballs and installers expect clippy to be present, but it's not always the case. For example, in try builds.
1 parent 6b06fdf commit 8a4a6f7

File tree

1 file changed

+29
-22
lines changed

1 file changed

+29
-22
lines changed

src/bootstrap/dist.rs

+29-22
Original file line numberDiff line numberDiff line change
@@ -1601,9 +1601,8 @@ impl Step for Extended {
16011601
prepare("cargo");
16021602
prepare("rust-analysis");
16031603
prepare("rust-std");
1604-
prepare("clippy");
16051604
prepare("rust-analyzer");
1606-
for tool in &["rust-docs", "rust-demangler", "miri"] {
1605+
for tool in &["clippy", "rust-docs", "rust-demangler", "miri"] {
16071606
if built_tools.contains(tool) {
16081607
prepare(tool);
16091608
}
@@ -1706,23 +1705,25 @@ impl Step for Extended {
17061705
.arg("-t")
17071706
.arg(etc.join("msi/remove-duplicates.xsl")),
17081707
);
1709-
builder.run(
1710-
Command::new(&heat)
1711-
.current_dir(&exe)
1712-
.arg("dir")
1713-
.arg("clippy")
1714-
.args(&heat_flags)
1715-
.arg("-cg")
1716-
.arg("ClippyGroup")
1717-
.arg("-dr")
1718-
.arg("Clippy")
1719-
.arg("-var")
1720-
.arg("var.ClippyDir")
1721-
.arg("-out")
1722-
.arg(exe.join("ClippyGroup.wxs"))
1723-
.arg("-t")
1724-
.arg(etc.join("msi/remove-duplicates.xsl")),
1725-
);
1708+
if built_tools.contains("clippy") {
1709+
builder.run(
1710+
Command::new(&heat)
1711+
.current_dir(&exe)
1712+
.arg("dir")
1713+
.arg("clippy")
1714+
.args(&heat_flags)
1715+
.arg("-cg")
1716+
.arg("ClippyGroup")
1717+
.arg("-dr")
1718+
.arg("Clippy")
1719+
.arg("-var")
1720+
.arg("var.ClippyDir")
1721+
.arg("-out")
1722+
.arg(exe.join("ClippyGroup.wxs"))
1723+
.arg("-t")
1724+
.arg(etc.join("msi/remove-duplicates.xsl")),
1725+
);
1726+
}
17261727
if built_tools.contains("rust-demangler") {
17271728
builder.run(
17281729
Command::new(&heat)
@@ -1806,14 +1807,16 @@ impl Step for Extended {
18061807
.arg("-dCargoDir=cargo")
18071808
.arg("-dStdDir=rust-std")
18081809
.arg("-dAnalysisDir=rust-analysis")
1809-
.arg("-dClippyDir=clippy")
18101810
.arg("-arch")
18111811
.arg(&arch)
18121812
.arg("-out")
18131813
.arg(&output)
18141814
.arg(&input);
18151815
add_env(builder, &mut cmd, target);
18161816

1817+
if built_tools.contains("clippy") {
1818+
cmd.arg("-dClippyDir=clippy");
1819+
}
18171820
if built_tools.contains("rust-docs") {
18181821
cmd.arg("-dDocsDir=rust-docs");
18191822
}
@@ -1840,7 +1843,9 @@ impl Step for Extended {
18401843
}
18411844
candle("CargoGroup.wxs".as_ref());
18421845
candle("StdGroup.wxs".as_ref());
1843-
candle("ClippyGroup.wxs".as_ref());
1846+
if built_tools.contains("clippy") {
1847+
candle("ClippyGroup.wxs".as_ref());
1848+
}
18441849
if built_tools.contains("miri") {
18451850
candle("MiriGroup.wxs".as_ref());
18461851
}
@@ -1877,9 +1882,11 @@ impl Step for Extended {
18771882
.arg("CargoGroup.wixobj")
18781883
.arg("StdGroup.wixobj")
18791884
.arg("AnalysisGroup.wixobj")
1880-
.arg("ClippyGroup.wixobj")
18811885
.current_dir(&exe);
18821886

1887+
if built_tools.contains("clippy") {
1888+
cmd.arg("ClippyGroup.wixobj");
1889+
}
18831890
if built_tools.contains("miri") {
18841891
cmd.arg("MiriGroup.wixobj");
18851892
}

0 commit comments

Comments
 (0)