From f80d5be76a6e9c2b045df3d111e037a59469da4c Mon Sep 17 00:00:00 2001 From: Zac Medico Date: Thu, 5 Jul 2007 01:17:32 +0000 Subject: Replace sys.exit() with return in depgraph.display() and action_build(). (trunk r7153) svn path=/main/branches/2.1.2/; revision=7154 --- bin/emerge | 32 ++++++++++++++++---------------- 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/bin/emerge b/bin/emerge index a50ac1e07..8657fea83 100755 --- a/bin/emerge +++ b/bin/emerge @@ -2751,7 +2751,7 @@ class depgraph: restrict = mydbapi.aux_get(pkg_key, ["RESTRICT"])[0] show_invalid_depstring_notice(x, restrict, str(e)) del e - sys.exit(1) + return 1 restrict = [] if "ebuild" == pkg_type and x[3] != "nomerge" and \ "fetch" in restrict: @@ -2904,7 +2904,7 @@ class depgraph: src_uri = portdb.aux_get(pkg_key, ["SRC_URI"])[0] show_invalid_depstring_notice(x, src_uri, str(e)) del e - sys.exit(1) + return 1 if myfilesdict is None: myfilesdict="[empty/missing/bad digest]" else: @@ -5267,7 +5267,7 @@ def action_build(settings, trees, mtimedb, else: if ("--resume" in myopts): print darkgreen("emerge: It seems we have nothing to resume...") - sys.exit(0) + return os.EX_OK myparams = create_depgraph_params(myopts, myaction) if myaction in ["system","world"]: @@ -5277,7 +5277,7 @@ def action_build(settings, trees, mtimedb, mydepgraph = depgraph(settings, trees, myopts, myparams, spinner) if not mydepgraph.xcreate(myaction): print "!!! Depgraph creation failed." - sys.exit(1) + return 1 if "--quiet" not in myopts and "--nodeps" not in myopts: print "\b\b... done!" else: @@ -5289,9 +5289,9 @@ def action_build(settings, trees, mtimedb, retval, favorites = mydepgraph.select_files(myfiles) except portage_exception.PackageNotFound, e: portage.writemsg("\n!!! %s\n" % str(e), noiselevel=-1) - sys.exit(1) + return 1 if not retval: - sys.exit(1) + return 1 if "--quiet" not in myopts and "--nodeps" not in myopts: print "\b\b... done!" @@ -5302,7 +5302,7 @@ def action_build(settings, trees, mtimedb, for x in mydepgraph.missingbins: sys.stderr.write(" "+str(x)+"\n") sys.stderr.write("\nThese are required by '--usepkgonly' -- Terminating.\n\n") - sys.exit(1) + return 1 if "--pretend" not in myopts and \ ("--ask" in myopts or "--tree" in myopts or \ @@ -5315,7 +5315,7 @@ def action_build(settings, trees, mtimedb, mymergelist = mymergelist[1:] if len(mymergelist) == 0: print colorize("INFORM", "emerge: It seems we have nothing to resume...") - sys.exit(0) + return os.EX_OK favorites = mtimedb["resume"]["favorites"] retval = mydepgraph.display(mymergelist, favorites=favorites) if retval != os.EX_OK: @@ -5337,7 +5337,7 @@ def action_build(settings, trees, mtimedb, print "!!! at the same time on the same system." if "--quiet" not in myopts: show_blocker_docs_link() - sys.exit(1) + return 1 if mergecount==0: if "--noreplace" in myopts and favorites: print @@ -5350,7 +5350,7 @@ def action_build(settings, trees, mtimedb, print print "Nothing to merge; quitting." print - sys.exit(0) + return os.EX_OK elif "--fetchonly" in myopts or "--fetch-all-uri" in myopts: prompt="Would you like to fetch the source files for these packages?" else: @@ -5360,7 +5360,7 @@ def action_build(settings, trees, mtimedb, print print "Quitting." print - sys.exit(0) + return os.EX_OK # Don't ask again (e.g. when auto-cleaning packages after merge) myopts.pop("--ask", None) @@ -5372,7 +5372,7 @@ def action_build(settings, trees, mtimedb, mymergelist = mymergelist[1:] if len(mymergelist) == 0: print colorize("INFORM", "emerge: It seems we have nothing to resume...") - sys.exit(0) + return os.EX_OK favorites = mtimedb["resume"]["favorites"] retval = mydepgraph.display(mymergelist, favorites=favorites) if retval != os.EX_OK: @@ -5387,13 +5387,13 @@ def action_build(settings, trees, mtimedb, not mydepgraph.digraph.hasallzeros(ignore_priority=DepPriority.MEDIUM): print "\n!!! --buildpkgonly requires all dependencies to be merged." print "!!! You have to merge the dependencies before you can build this package.\n" - sys.exit(1) + return 1 else: if ("--buildpkgonly" in myopts): if not mydepgraph.digraph.hasallzeros(ignore_priority=DepPriority.MEDIUM): print "\n!!! --buildpkgonly requires all dependencies to be merged." print "!!! Cannot merge requested packages. Merge deps and try again.\n" - sys.exit(1) + return 1 if ("--resume" in myopts): favorites=mtimedb["resume"]["favorites"] @@ -5407,7 +5407,7 @@ def action_build(settings, trees, mtimedb, retval = mergetask.merge( mtimedb["resume"]["mergelist"], favorites, mtimedb) if retval != os.EX_OK: - sys.exit(retval) + return retval else: if "resume" in mtimedb and \ "mergelist" in mtimedb["resume"] and \ @@ -5446,7 +5446,7 @@ def action_build(settings, trees, mtimedb, mergetask = MergeTask(settings, trees, myopts) retval = mergetask.merge(pkglist, favorites, mtimedb) if retval != os.EX_OK: - sys.exit(retval) + return retval if mtimedb.has_key("resume"): del mtimedb["resume"] -- cgit v1.2.3-1-g7c22