diff options
author | Zac Medico <zmedico@gentoo.org> | 2008-03-28 12:28:25 +0000 |
---|---|---|
committer | Zac Medico <zmedico@gentoo.org> | 2008-03-28 12:28:25 +0000 |
commit | ef2adcd146410fecbc0edbbc87cdbaca95ac7fd8 (patch) | |
tree | d44b4cf2102ef8989a14ba568a2bb097eaf7b737 /bin/dohtml | |
parent | 3eb52bb2f060fd9a618284be42f11cc810ad59e2 (diff) | |
download | portage-ef2adcd146410fecbc0edbbc87cdbaca95ac7fd8.tar.gz portage-ef2adcd146410fecbc0edbbc87cdbaca95ac7fd8.tar.bz2 portage-ef2adcd146410fecbc0edbbc87cdbaca95ac7fd8.zip |
Only show the eqawarn from bug #149745 when nothing is found to install. Also,
exit successfully unless nothing is found to install. (trunk r9484)
svn path=/main/branches/2.1.2/; revision=9562
Diffstat (limited to 'bin/dohtml')
-rwxr-xr-x | bin/dohtml | 19 |
1 files changed, 12 insertions, 7 deletions
diff --git a/bin/dohtml b/bin/dohtml index 23d0292f4..6162b0199 100755 --- a/bin/dohtml +++ b/bin/dohtml @@ -46,6 +46,8 @@ def eqawarn(lines): cmd += "eqawarn \"%s\" ; " % line os.spawnlp(os.P_WAIT, "bash", "bash", "-c", cmd) +skipped_directories = [] + def install(basename, dirname, options, prefix=""): fullpath = basename if prefix: @@ -70,8 +72,8 @@ def install(basename, dirname, options, prefix=""): if prefix: pfx = prefix + "/" + pfx install(i, dirname, options, pfx) elif not options.recurse and os.path.isdir(fullpath): - eqawarn(["QA Notice: dohtml on directory " + \ - "'%s' without recursion option" % fullpath]) + global skipped_directories + skipped_directories.append(fullpath) return False else: return False @@ -166,19 +168,22 @@ def main(): print "Document prefix : '" + options.doc_prefix + "'" print "Allowed files :", options.allowed_files - success = True + success = False for x in args: basename = os.path.basename(x) dirname = os.path.dirname(x) - if not install(basename, dirname, options): - success = False - + success |= install(basename, dirname, options) + if success: retcode = 0 else: retcode = 1 - + global skipped_directories + for x in skipped_directories: + eqawarn(["QA Notice: dohtml on directory " + \ + "'%s' without recursion option" % x]) + sys.exit(retcode) if __name__ == "__main__": |