diff options
author | Dan Willemsen <dwillemsen@google.com> | 2017-07-13 14:46:27 -0700 |
---|---|---|
committer | Dan Willemsen <dwillemsen@google.com> | 2017-07-13 15:09:44 -0700 |
commit | 38cef8ac3979da8e13037540981fe7aecc1e1fbe (patch) | |
tree | 5f42cecf00edd52907a2ad79399f0f198cde6676 | |
parent | cae59bc29de92e135ca53cfb1142d3524b62f846 (diff) | |
download | build_soong-38cef8ac3979da8e13037540981fe7aecc1e1fbe.tar.gz build_soong-38cef8ac3979da8e13037540981fe7aecc1e1fbe.tar.bz2 build_soong-38cef8ac3979da8e13037540981fe7aecc1e1fbe.zip |
microfactory: Sort dependencies
This was causing android/soong/ui/build to rebuild sometimes when it
didn't need to.
Test: `m -j nothing; grep "B compile" out/.soong_ui.trace` repeatedly
Change-Id: I5d33271a7b037f53674a0d312d8da1746eed8eaa
-rw-r--r-- | cmd/microfactory/microfactory.go | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/cmd/microfactory/microfactory.go b/cmd/microfactory/microfactory.go index 18008da2..0f621f2b 100644 --- a/cmd/microfactory/microfactory.go +++ b/cmd/microfactory/microfactory.go @@ -230,6 +230,7 @@ func (p *GoPackage) findDeps(path string, pkgMap *pkgPathMapping, allPackages *l fmt.Fprintf(os.Stderr, "Package %q depends on %v\n", p.Name, deps) } + sort.Strings(deps) for _, dep := range deps { p.directDeps = append(p.directDeps, allPackages.getByName(dep)) } |