Mirrors/rust-analyzer
2
0
Fork 0
mirror of https://github.com/rust-lang/rust-analyzer synced 2024-12-26 21:13:37 +00:00
Code Issues Projects Releases Packages Wiki Activity

fix: Don't emit --keep-going for custom build script commands

Browse source
This commit is contained in:
Lukas Wirth 2024-05-14 21:26:37 +02:00
parent 465ebbc102
commit eac2e512a1
2 changed files with 16 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
crates/project-model/src/build_scripts.rs
View file

@ -64,8 +64,10 @@ impl WorkspaceBuildScripts {
config: &CargoConfig,
allowed_features: &FxHashSet<String>,
manifest_path: &ManifestPath,
toolchain: Option<&Version>,
sysroot: Option<&Sysroot>,
) -> io::Result<Command> {
const RUST_1_75: Version = Version::new(1, 75, 0);
let mut cmd = match config.run_build_script_command.as_deref() {
Some([program, args @ ..]) => {
let mut cmd = Command::new(program);
@ -120,6 +122,10 @@ impl WorkspaceBuildScripts {
cmd.arg("-Zscript");
}
if toolchain.map_or(false, |it| *it >= RUST_1_75) {
cmd.arg("--keep-going");
}
cmd
}
};
@ -142,11 +148,9 @@ impl WorkspaceBuildScripts {
config: &CargoConfig,
workspace: &CargoWorkspace,
progress: &dyn Fn(String),
toolchain: &Option<Version>,
toolchain: Option<&Version>,
sysroot: Option<&Sysroot>,
) -> io::Result<WorkspaceBuildScripts> {
const RUST_1_75: Version = Version::new(1, 75, 0);
let current_dir = match &config.invocation_location {
InvocationLocation::Root(root) if config.run_build_script_command.is_some() => {
root.as_path()
@ -156,11 +160,13 @@ impl WorkspaceBuildScripts {
.as_ref();
let allowed_features = workspace.workspace_features();
let mut cmd =
Self::build_command(config, &allowed_features, workspace.manifest_path(), sysroot)?;
if toolchain.as_ref().map_or(false, |it| *it >= RUST_1_75) {
cmd.args(["--keep-going"]);
}
let cmd = Self::build_command(
config,
&allowed_features,
workspace.manifest_path(),
toolchain,
sysroot,
)?;
Self::run_per_ws(cmd, workspace, current_dir, progress)
}
@ -189,6 +195,7 @@ impl WorkspaceBuildScripts {
// This is not gonna be used anyways, so just construct a dummy here
&ManifestPath::try_from(workspace_root.clone()).unwrap(),
None,
None,
)?;
// NB: Cargo.toml could have been modified between `cargo metadata` and
// `cargo check`. We shouldn't assume that package ids we see here are

2
crates/project-model/src/workspace.rs
View file

@ -488,7 +488,7 @@ impl ProjectWorkspace {
config,
cargo,
progress,
&self.toolchain,
self.toolchain.as_ref(),
self.sysroot.as_ref().ok(),
)
.with_context(|| {