Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1647)

Unified Diff: parallel_emerge

Issue 3308027: Fix rebuild blacklist so that icedtea isn't rebuilt unnecessarily. (Closed) Base URL: http://git.chromium.org/git/crosutils.git
Patch Set: Created 10 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: parallel_emerge
diff --git a/parallel_emerge b/parallel_emerge
index c53d765e0eb7512b6912ab3f6b9dbaa65cf4b869..da2d058660ce62b570dba9af1d87aaf4675f2077 100755
--- a/parallel_emerge
+++ b/parallel_emerge
@@ -1016,13 +1016,13 @@ class DepGraphGenerator(object):
# built from source, all children must also be built from source.
local_ready_cache, remote_ready_cache = {}, {}
local_mtime_cache, remote_mtime_cache = {}, {}
- for pkg in final_pkgs:
+ for pkg in final_pkgs.difference(rebuild_blacklist):
# If all the necessary local packages are ready, and their
# modification times are in sync, we don't need to do anything here.
local_mtime = LastModifiedWithDeps(pkg, local_pkgs, local_mtime_cache)
local_ready = PrebuiltsReady(pkg, local_pkgs, local_ready_cache)
if (not local_ready or local_pkgs.get(pkg, 0) < local_mtime and
- pkg not in cycles and pkg not in rebuild_blacklist):
+ pkg not in cycles):
# OK, at least one package is missing from the local cache or is
# outdated. This means we're going to have to install the package
# and all dependencies.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698