nordic-dev.net/rust
nordic-dev.net/rust
2
0
Fork 0
mirror of https://github.com/rust-lang/rust.git synced 2025-01-20 19:52:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Unique package by name and version.

Browse Source 
This commit is a fixup of a bug I introduced by using a PackageId to refer to a crate when its name conflicts with a dependency.
It turns out the package id currently is `name version path` while cargo expects `name:version` as argument.
This commit is contained in:
o0Ignition0o 2020-04-02 09:00:44 +02:00
parent f696df379a
commit f643b4b5f5
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
crates/ra_project_model/src/cargo_workspace.rs
View File

@ -75,7 +75,7 @@ pub type Target = Idx<TargetData>;
#[derive(Debug, Clone)]
pub struct PackageData {
pub id: String,
pub version: String,
pub name: String,
pub manifest: PathBuf,
pub targets: Vec<Target>,
@ -174,14 +174,15 @@ impl CargoWorkspace {
let ws_members = &meta.workspace_members;
for meta_pkg in meta.packages {
let cargo_metadata::Package { id, edition, name, manifest_path, .. } = meta_pkg;
let cargo_metadata::Package { id, edition, name, manifest_path, version, .. } =
meta_pkg;
let is_member = ws_members.contains(&id);
let edition = edition
.parse::<Edition>()
.with_context(|| format!("Failed to parse edition {}", edition))?;
let pkg = packages.alloc(PackageData {
name,
id: id.to_string(),
version: version.to_string(),
manifest: manifest_path,
targets: Vec::new(),
is_member,
@ -256,7 +257,7 @@ impl CargoWorkspace {
if self.is_unique(&*package.name) {
package.name.clone()
} else {
package.id.clone()
format!("{}:{}", package.name, package.version)
}
}