language-servers/rust-analyzer
1
0
Fork 0
mirror of https://github.com/rust-lang/rust-analyzer.git synced 2025-10-31 12:04:43 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Cleanup unstable flags handling

Browse source
This commit is contained in:
Lukas Wirth 2025-07-27 21:22:50 +02:00
parent df85aac1d4
commit b398bc6af7
3 changed files with 18 additions and 19 deletions
Download patch file Download diff file Expand all files Collapse all files

9
crates/project-model/src/build_dependencies.rs
View file

@ -62,7 +62,10 @@ impl BuildScriptOutput {
self.cfgs.is_empty()
&& self.envs.is_empty()
&& self.out_dir.is_none()
&& self.proc_macro_dylib_path == ProcMacroDylibPath::NotBuilt
&& matches!(
self.proc_macro_dylib_path,
ProcMacroDylibPath::NotBuilt | ProcMacroDylibPath::NotProcMacro
)
}
}
@ -462,10 +465,10 @@ impl WorkspaceBuildScripts {
let lockfile_path =
<_ as AsRef<Utf8Path>>::as_ref(manifest_path).with_extension("lock");
if let Some((temp_dir, target_lockfile)) = make_lockfile_copy(&lockfile_path) {
requires_unstable_options = true;
temp_dir_guard = Some(temp_dir);
cmd.arg("--lockfile-path");
cmd.arg(target_lockfile.as_str());
requires_unstable_options = true;
}
}
match &config.features {
@ -499,7 +502,7 @@ impl WorkspaceBuildScripts {
// available in current toolchain's cargo, use it to build compile time deps only.
const COMP_TIME_DEPS_MIN_TOOLCHAIN_VERSION: semver::Version = semver::Version {
major: 1,
minor: 189,
minor: 89,
patch: 0,
pre: semver::Prerelease::EMPTY,
build: semver::BuildMetadata::EMPTY,

9
crates/project-model/src/cargo_workspace.rs
View file

@ -601,7 +601,6 @@ impl FetchMetadata {
}
command.current_dir(current_dir);
let mut needs_nightly = false;
let mut other_options = vec![];
// cargo metadata only supports a subset of flags of what cargo usually accepts, and usually
// the only relevant flags for metadata here are unstable ones, so we pass those along
@ -611,7 +610,6 @@ impl FetchMetadata {
if arg == "-Z"
&& let Some(arg) = extra_args.next()
{
needs_nightly = true;
other_options.push("-Z".to_owned());
other_options.push(arg.to_owned());
}
@ -619,7 +617,6 @@ impl FetchMetadata {
let mut lockfile_path = None;
if cargo_toml.is_rust_manifest() {
needs_nightly = true;
other_options.push("-Zscript".to_owned());
} else if config
.toolchain_version
@ -637,10 +634,6 @@ impl FetchMetadata {
command.other_options(other_options.clone());
if needs_nightly {
command.env("__CARGO_TEST_CHANNEL_OVERRIDE_DO_NOT_USE_THIS", "nightly");
}
// Pre-fetch basic metadata using `--no-deps`, which:
// - avoids fetching registries like crates.io,
// - skips dependency resolution and does not modify lockfiles,
@ -710,7 +703,7 @@ impl FetchMetadata {
other_options.push(target_lockfile.to_string());
using_lockfile_copy = true;
}
if using_lockfile_copy {
if using_lockfile_copy || other_options.iter().any(|it| it.starts_with("-Z")) {
command.env("__CARGO_TEST_CHANNEL_OVERRIDE_DO_NOT_USE_THIS", "nightly");
other_options.push("-Zunstable-options".to_owned());
}

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

@ -1316,15 +1316,18 @@ fn cargo_to_crate_graph(
public_deps.add_to_crate_graph(crate_graph, from);
// Add dep edge of all targets to the package's lib target
if let Some((to, name)) = lib_tgt.clone() {
match to != from && kind != TargetKind::BuildScript {
true => {
if let Some((to, name)) = lib_tgt.clone()
&& to != from
&& kind != TargetKind::BuildScript
{
// (build script can not depend on its library target)
// For root projects with dashes in their name,
// cargo metadata does not do any normalization,
// so we do it ourselves currently
let name = CrateName::normalize_dashes(&name);
add_dep(crate_graph, from, name, to);
}
false => (),
}
}
}
}