Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

Commit

Permalink
status: use IgnoredPackages() to get all packages
Browse files Browse the repository at this point in the history
This change removes the usage of IgnoredPackages() separately and passes
the same to ToReachMap(), which returns the packages with all ignoring
rules applied.
  • Loading branch information
darkowlzz committed Sep 28, 2017
1 parent f5750a7 commit 7c5da12
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions cmd/dep/status.go
Original file line number Diff line number Diff line change
Expand Up @@ -566,7 +566,7 @@ func runStatusAll(ctx *dep.Ctx, out outputter, p *dep.Project, sm gps.SourceMana
//
// It's possible for digests to not match, but still have a correct
// lock.
rm, _ := ptree.ToReachMap(true, true, false, nil)
rm, _ := ptree.ToReachMap(true, true, false, p.Manifest.IgnoredPackages())

external := rm.FlattenFn(paths.IsStandardImportPath)
roots := make(map[gps.ProjectRoot][]string, len(external))
Expand Down Expand Up @@ -603,9 +603,6 @@ func runStatusAll(ctx *dep.Ctx, out outputter, p *dep.Project, sm gps.SourceMana

out.MissingHeader()

// Ignored packages.
igPkgs := p.Manifest.IgnoredPackages()

outer:
for root, pkgs := range roots {
// TODO also handle the case where the project is present, but there
Expand All @@ -616,10 +613,6 @@ outer:
}
}

if _, ok := igPkgs[string(root)]; ok {
continue outer
}

hasMissingPkgs = true
out.MissingLine(&MissingStatus{ProjectRoot: string(root), MissingPackages: pkgs})
}
Expand Down

0 comments on commit 7c5da12

Please sign in to comment.