[patch] in case of blockers, bail out earlier to avoid later problems

blockers.patch (text/plain), 1.97 KB, created by Zac Medico on 2007-12-13 01:27:04 UTC

Index: bin/emerge
===================================================================
--- bin/emerge	(revision 8889)
+++ bin/emerge	(revision 8890)
@@ -3689,19 +3689,6 @@
 		ldpath_mtimes = mtimedb["ldpath"]
 		xterm_titles = "notitles" not in self.settings.features
 
-		#check for blocking dependencies
-		if "--fetchonly" not in self.myopts and \
-			"--fetch-all-uri" not in self.myopts and \
-			"--buildpkgonly" not in self.myopts:
-			for x in mylist:
-				if x[0]=="blocks":
-					print "\n!!! Error: the "+x[2]+" package conflicts with another package;"
-					print   "!!!        the two packages cannot be installed on the same system together."
-					print   "!!!        Please use 'emerge --pretend' to determine blockers."
-					if "--quiet" not in self.myopts:
-						show_blocker_docs_link()
-					return 1
-
 		if "--resume" in self.myopts:
 			# We're resuming.
 			print colorize("GOOD", "*** Resuming merge...")
@@ -6043,13 +6030,27 @@
 							("--pretend" in myopts),
 							mydbapi=trees[pkgline[1]]["porttree"].dbapi,
 							tree="porttree")
-			if "--fetchonly" in myopts or "--fetch-all-uri" in myopts:
-				pkglist = []
-				for pkg in mydepgraph.altlist():
-					if pkg[0] != "blocks":
-						pkglist.append(pkg)
+
+			pkglist = mydepgraph.altlist()
+
+			if fetchonly or "--buildpkgonly"  in myopts:
+				pkglist = [pkg for pkg in pkglist if pkg[0] != "blocks"]
 			else:
-				pkglist = mydepgraph.altlist()
+				for x in pkglist:
+					if x[0] != "blocks":
+						continue
+					msg = "Error: the " + x[2] + " package conflicts " + \
+					"with another package; the two packages cannot " + \
+					"be installed on the same system together. " + \
+					"Please use 'emerge --pretend' to determine blockers."
+					prefix = red(" * ")
+					from textwrap import wrap
+					for line in wrap(msg, 70):
+						print prefix + line
+					if "--quiet" not in myopts:
+						show_blocker_docs_link()
+					return 1
+
 			if favorites:
 				mydepgraph.saveNomergeFavorites()
 			del mydepgraph