Skip to content

Commit

Permalink
Merge pull request #147 from danhhz/exclude
Browse files Browse the repository at this point in the history
make --exclude also exclude packages from results
  • Loading branch information
est31 committed Jan 12, 2023
2 parents 6859332 + 620c9b2 commit b84d478
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions src/lib.rs
Expand Up @@ -477,12 +477,21 @@ impl OptUdeps {

let mut outcome = Outcome::default();

let included_packages = compile_opts.spec.get_packages(&ws)?
.iter()
.map(|x|x.package_id())
.collect::<HashSet<_>>();
for (dependencies, used_dependencies, kind) in &[
(&normal_dependencies, &used_normal_dev_dependencies, dependency::DepKind::Normal),
(&dev_dependencies, &used_normal_dev_dependencies, dependency::DepKind::Development),
(&build_dependencies, &used_build_dependencies, dependency::DepKind::Build),
] {
for &(id, dependency) in *dependencies {
// This package may have been explictly excluded via flags.
if !included_packages.contains(&id) {
continue;
}

use anyhow::Context;
let ignore = ws_resolve
.pkg_set
Expand Down

0 comments on commit b84d478

Please sign in to comment.