From 6f03cce1c3e6083f882fc56100487e616f60fa6b Mon Sep 17 00:00:00 2001 From: Ray Tung Date: Wed, 4 Oct 2017 00:50:33 +1100 Subject: [PATCH] Removed root related flags --- src/cargo/core/resolver/encode.rs | 25 +------------------------ src/cargo/ops/lockfile.rs | 1 - 2 files changed, 1 insertion(+), 25 deletions(-) diff --git a/src/cargo/core/resolver/encode.rs b/src/cargo/core/resolver/encode.rs index 344082bf6..2ccb1a7c9 100644 --- a/src/cargo/core/resolver/encode.rs +++ b/src/cargo/core/resolver/encode.rs @@ -14,8 +14,6 @@ use super::Resolve; #[derive(Serialize, Deserialize, Debug)] pub struct EncodableResolve { package: Option>, - /// `root` is optional to allow forward compatibility. - root: Option, metadata: Option, #[serde(default, skip_serializing_if = "Patch::is_empty")] @@ -33,13 +31,7 @@ impl EncodableResolve { pub fn into_resolve(self, ws: &Workspace) -> CargoResult { let path_deps = build_path_deps(ws); - let packages = { - let mut packages = self.package.unwrap_or_default(); - if let Some(root) = self.root { - packages.insert(0, root); - } - packages - }; + let packages = self.package.unwrap_or_default(); // `PackageId`s in the lock file don't include the `source` part // for workspace members, so we reconstruct proper ids. @@ -311,7 +303,6 @@ impl<'de> de::Deserialize<'de> for EncodablePackageId { pub struct WorkspaceResolve<'a, 'cfg: 'a> { pub ws: &'a Workspace<'cfg>, pub resolve: &'a Resolve, - pub use_root_key: bool, } impl<'a, 'cfg> ser::Serialize for WorkspaceResolve<'a, 'cfg> { @@ -321,15 +312,7 @@ impl<'a, 'cfg> ser::Serialize for WorkspaceResolve<'a, 'cfg> { let mut ids: Vec<&PackageId> = self.resolve.graph.iter().collect(); ids.sort(); - let root = self.ws.members().max_by_key(|member| { - member.name() - }).map(Package::package_id); - let encodable = ids.iter().filter_map(|&id| { - if self.use_root_key && root.unwrap() == id { - return None - } - Some(encodable_resolve_node(id, self.resolve)) }).collect::>(); @@ -347,11 +330,6 @@ impl<'a, 'cfg> ser::Serialize for WorkspaceResolve<'a, 'cfg> { let metadata = if metadata.is_empty() { None } else { Some(metadata) }; - let root = match root { - Some(root) if self.use_root_key => Some(encodable_resolve_node(root, self.resolve)), - _ => None, - }; - let patch = Patch { unused: self.resolve.unused_patches().iter().map(|id| { EncodableDependency { @@ -365,7 +343,6 @@ impl<'a, 'cfg> ser::Serialize for WorkspaceResolve<'a, 'cfg> { }; EncodableResolve { package: Some(encodable), - root: root, metadata: metadata, patch: patch, }.serialize(s) diff --git a/src/cargo/ops/lockfile.rs b/src/cargo/ops/lockfile.rs index 6fa4e2f71..8dc5b4567 100644 --- a/src/cargo/ops/lockfile.rs +++ b/src/cargo/ops/lockfile.rs @@ -43,7 +43,6 @@ pub fn write_pkg_lockfile(ws: &Workspace, resolve: &Resolve) -> CargoResult<()> let toml = toml::Value::try_from(WorkspaceResolve { ws: ws, resolve: resolve, - use_root_key: false, }).unwrap(); let mut out = String::new(); -- 2.30.2