buildhistory-collect-srcrevs: match new buildhistory-diff command line parsing
* Default buildhistory directory to buildhistory/ under the current directory and require an option to set it * Show a description in the help output (From OE-Core rev: 64aa7d0b53f6ad45ab1a2121e917d7a512097407) Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
parent
2a9d9b6a99
commit
752df39cd2
|
@ -34,8 +34,9 @@ logger = logger_create()
|
||||||
|
|
||||||
def main():
|
def main():
|
||||||
parser = optparse.OptionParser(
|
parser = optparse.OptionParser(
|
||||||
|
description = "Collects the recorded SRCREV values from buildhistory and reports on them.",
|
||||||
usage = """
|
usage = """
|
||||||
%prog [options] <buildhistory-dir>""")
|
%prog [options]""")
|
||||||
|
|
||||||
parser.add_option("-a", "--report-all",
|
parser.add_option("-a", "--report-all",
|
||||||
help = "Report all SRCREV values, not just ones where AUTOREV has been used",
|
help = "Report all SRCREV values, not just ones where AUTOREV has been used",
|
||||||
|
@ -43,16 +44,20 @@ def main():
|
||||||
parser.add_option("-f", "--forcevariable",
|
parser.add_option("-f", "--forcevariable",
|
||||||
help = "Use forcevariable override for all output lines",
|
help = "Use forcevariable override for all output lines",
|
||||||
action="store_true", dest="forcevariable")
|
action="store_true", dest="forcevariable")
|
||||||
|
parser.add_option("-p", "--buildhistory-dir",
|
||||||
|
help = "Specify path to buildhistory directory (defaults to buildhistory/ under cwd)",
|
||||||
|
action="store", dest="buildhistory_dir", default='buildhistory/')
|
||||||
|
|
||||||
options, args = parser.parse_args(sys.argv)
|
options, args = parser.parse_args(sys.argv)
|
||||||
|
|
||||||
if len(args) != 2:
|
if len(args) > 1:
|
||||||
|
sys.stderr.write('Invalid argument(s) specified: %s\n\n' % ' '.join(args[1:]))
|
||||||
parser.print_help()
|
parser.print_help()
|
||||||
sys.exit(1)
|
sys.exit(1)
|
||||||
|
|
||||||
buildhistory_dir = args[1]
|
if not os.path.exists(options.buildhistory_dir):
|
||||||
if not os.path.exists(buildhistory_dir):
|
sys.stderr.write('Buildhistory directory "%s" does not exist\n\n' % options.buildhistory_dir)
|
||||||
logger.error('specified buildhistory path %s could not be found' % buildhistory_dir)
|
parser.print_help()
|
||||||
sys.exit(1)
|
sys.exit(1)
|
||||||
|
|
||||||
if options.forcevariable:
|
if options.forcevariable:
|
||||||
|
@ -61,7 +66,7 @@ def main():
|
||||||
forcevariable = ''
|
forcevariable = ''
|
||||||
|
|
||||||
lastdir = ''
|
lastdir = ''
|
||||||
for root, dirs, files in os.walk(buildhistory_dir):
|
for root, dirs, files in os.walk(options.buildhistory_dir):
|
||||||
if '.git' in dirs:
|
if '.git' in dirs:
|
||||||
dirs.remove('.git')
|
dirs.remove('.git')
|
||||||
for fn in files:
|
for fn in files:
|
||||||
|
|
Loading…
Reference in New Issue