From cfa8c4c33ceae77cf8e1e6be25df552c7431017c Mon Sep 17 00:00:00 2001
From: Zac Medico <zmedico@gentoo.org>
Date: Wed, 14 Jun 2006 10:08:51 +0000
Subject: Give a useful error message when a dependency has an ambiguous atom
 due to the category being unspecified.  See bug #123677.

svn path=/main/trunk/; revision=3509
---
 bin/emerge | 53 ++++++++++++++++++++++++++++++++++++-----------------
 1 file changed, 36 insertions(+), 17 deletions(-)

diff --git a/bin/emerge b/bin/emerge
index cb93c4e36..4809f5ffe 100755
--- a/bin/emerge
+++ b/bin/emerge
@@ -1023,23 +1023,42 @@ class depgraph:
 		mydep={}
 		mp=string.join(mybigkey)
 
-		if myroot=="/":
-			mydep["/"]=edepend["DEPEND"]+" "+edepend["RDEPEND"]
-			if not self.select_dep("/",mydep["/"],myparent=mp,myuse=myuse):
-				return 0
-		else:
-			mydep["/"]=edepend["DEPEND"]
-			mydep[myroot]=edepend["RDEPEND"]
-			if not self.select_dep("/",mydep["/"],myparent=mp,myuse=myuse):
-				return 0
-			if not self.select_dep(myroot,mydep[myroot],myparent=mp,myuse=myuse):
-				return 0
-
-		if edepend.has_key("PDEPEND") and edepend["PDEPEND"]:
-			# Post Depend -- Add to the list without a parent, as it depends
-			# on a package being present AND must be built after that package.
-			if not self.select_dep(myroot,edepend["PDEPEND"],myuse=myuse):
-				return 0
+		try:
+			if myroot=="/":
+				mydep["/"]=edepend["DEPEND"]+" "+edepend["RDEPEND"]
+				if not self.select_dep("/",mydep["/"],myparent=mp,myuse=myuse):
+					return 0
+			else:
+				mydep["/"]=edepend["DEPEND"]
+				mydep[myroot]=edepend["RDEPEND"]
+				if not self.select_dep("/",mydep["/"],myparent=mp,myuse=myuse):
+					return 0
+				if not self.select_dep(myroot,mydep[myroot],myparent=mp,myuse=myuse):
+					return 0
+	
+			if edepend.has_key("PDEPEND") and edepend["PDEPEND"]:
+				# Post Depend -- Add to the list without a parent, as it depends
+				# on a package being present AND must be built after that package.
+				if not self.select_dep(myroot,edepend["PDEPEND"],myuse=myuse):
+					return 0
+		except ValueError, e:
+			pkgs = e.args[0]
+			portage.writemsg("\n\n!!! An atom in the dependencies " + \
+				"is not fully-qualified. Multiple matches:\n\n", noiselevel=-1)
+			for cpv in pkgs:
+				portage.writemsg("    %s\n" % cpv, noiselevel=-1)
+			portage.writemsg("\n", noiselevel=-1)
+			if mytype == "binary":
+				portage.writemsg(
+					"!!! This binary package cannot be installed: '%s'\n" % \
+					mykey, noiselevel=-1)
+			elif mytype == "ebuild":
+				myebuild, mylocation = portage.portdb.findname2(mykey)
+				portage.writemsg("!!! This ebuild cannot be installed: " + \
+					"'%s'\n" % myebuild, noiselevel=-1)
+			portage.writemsg("!!! Please notify the package maintainer " + \
+				"that atoms must be fully-qualified.\n", noiselevel=-1)
+			return 0
 
 		return 1
 
-- 
cgit v1.2.3-1-g7c22