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

Unified Diff: parallel_emerge

Issue 3029051: Blacklist icedtea and o3d from automatic rebuilds. (Closed) Base URL: ssh://git@chromiumos-git/crosutils.git
Patch Set: Created 10 years, 5 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 5c7d82c37331cb3ee5ca43894015500e58389bf8..e5920f4fce6c116150cf5d52ee00d9075623eb62 100755
--- a/parallel_emerge
+++ b/parallel_emerge
@@ -557,6 +557,14 @@ class DepGraphGenerator(object):
# we're done. It's populated in BuildFinalPackageSet()
final_pkgs = set()
+ # These packages take a really long time to build, so, for expediency, we
+ # are blacklisting them from automatic rebuilds. Instead, these packages
+ # will only be rebuilt when they are explicitly rev'd.
+ rebuild_blacklist = set()
+ for pkg in ("media-plugins/o3d", "dev-java/icedtea"):
+ for match in final_db.match_pkgs(pkg):
+ rebuild_blacklist.add(str(match.cpv))
+
# deps_map is the actual dependency graph.
#
# Each package specifies a "needs" list and a "provides" list. The "needs"
@@ -746,8 +754,9 @@ class DepGraphGenerator(object):
"""Merge this package and all packages it provides."""
this_pkg = deps_map[pkg]
- if this_pkg[merge_type] or pkg not in final_pkgs:
- return set()
+ if (this_pkg[merge_type] or pkg not in final_pkgs or
+ pkg in rebuild_blacklist):
+ return
# Mark this package as non-optional
deps_info[pkg]["optional"] = False
« 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