For bug #180165, make portageq print a "Permission denied" error when appropriate and make has_version die if portageq has an unexpected exit code. (trunk r6880:6882)

svn path=/main/branches/2.1.2/; revision=6885

3 files changed, 23 insertions, 7 deletions

diff --git a/bin/ebuild.sh b/bin/ebuild.sh
index 029d46a2f..6ea33964c 100755
--- a/bin/ebuild.sh
+++ b/bin/ebuild.sh
@@ -190,11 +190,19 @@ has_version() {
 	fi
 	# return shell-true/shell-false if exists.
 	# Takes single depend-type atoms.
-	if "${PORTAGE_BIN_PATH}/portageq" 'has_version' "${ROOT}" "$1"; then
-		return 0
-	else
-		return 1
-	fi
+	"${PORTAGE_BIN_PATH}"/portageq has_version "${ROOT}" "$1"
+	local retval=$?
+	case "${retval}" in
+		0)
+			return 0
+			;;
+		1)
+			return 1
+			;;
+		*)
+			die "unexpected portageq exit code: ${retval}"
+			;;
+	esac
 }
 
 portageq() {
diff --git a/bin/emerge b/bin/emerge
index 27a48ee2c..0084dded9 100755
--- a/bin/emerge
+++ b/bin/emerge
@@ -5582,5 +5582,10 @@ def emerge_main():
 			post_emerge(settings, mtimedb, 0)
 
 if __name__ == "__main__":
-	retval = emerge_main()
+	from portage_exception import PermissionDenied
+	try:
+		retval = emerge_main()
+	except PermissionDenied, e:
+		sys.stderr.write("Permission denied: '%s'\n" % str(e))
+		sys.exit(e.errno)
 	sys.exit(retval)
diff --git a/bin/portageq b/bin/portageq
index de2a24c12..1a59af8bc 100755
--- a/bin/portageq
+++ b/bin/portageq
@@ -22,7 +22,7 @@ except KeyboardInterrupt:
 import os
 
 import types
-
+import portage_exception
 #-----------------------------------------------------------------------------
 #
 # To add functionality to this tool, add a function below.
@@ -342,6 +342,9 @@ def main():
 	except KeyError:
 		usage(sys.argv)
 		sys.exit(os.EX_USAGE)
+	except portage_exception.PermissionDenied, e:
+		sys.stderr.write("Permission denied: '%s'\n" % str(e))
+		sys.exit(e.errno)
 
 main()