diff options
author | Ross Burton <ross@burtonini.com> | 2020-09-15 13:55:48 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2020-09-17 09:47:46 +0100 |
commit | 4064756e20bb4eff59d53f495b89d16ff479bb4d (patch) | |
tree | 6c6dabc0c50d22f6912fcd61d82fa2cea8e80474 | |
parent | d393196485d490af8397d074e2060540ff41c504 (diff) | |
download | poky-4064756e20bb4eff59d53f495b89d16ff479bb4d.tar.gz |
buildhistory-diff: use BUILDDIR to know where buildhistory is
Instead of assuming that the buildhistory is stored in ./buildhistory
unless told otherwise, use the environment variable BUILDDIR if present
(which is set by oe-init-build-env) to locate the buildhistory
directory.
(From OE-Core rev: 18ca377df98e53b84ae8ec9ba561392c776c10df)
Signed-off-by: Ross Burton <ross.burton@arm.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rwxr-xr-x | scripts/buildhistory-diff | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/scripts/buildhistory-diff b/scripts/buildhistory-diff index 833f7c33a5..3bd40a2a1e 100755 --- a/scripts/buildhistory-diff +++ b/scripts/buildhistory-diff | |||
@@ -28,10 +28,12 @@ def get_args_parser(): | |||
28 | %(prog)s [options] [from-revision [to-revision]] | 28 | %(prog)s [options] [from-revision [to-revision]] |
29 | (if not specified, from-revision defaults to build-minus-1, and to-revision defaults to HEAD)""") | 29 | (if not specified, from-revision defaults to build-minus-1, and to-revision defaults to HEAD)""") |
30 | 30 | ||
31 | default_dir = os.path.join(os.environ.get('BUILDDIR', '.'), 'buildhistory') | ||
32 | |||
31 | parser.add_argument('-p', '--buildhistory-dir', | 33 | parser.add_argument('-p', '--buildhistory-dir', |
32 | action='store', | 34 | action='store', |
33 | dest='buildhistory_dir', | 35 | dest='buildhistory_dir', |
34 | default='buildhistory/', | 36 | default=default_dir, |
35 | help="Specify path to buildhistory directory (defaults to buildhistory/ under cwd)") | 37 | help="Specify path to buildhistory directory (defaults to buildhistory/ under cwd)") |
36 | parser.add_argument('-v', '--report-version', | 38 | parser.add_argument('-v', '--report-version', |
37 | action='store_true', | 39 | action='store_true', |
@@ -80,11 +82,6 @@ def main(): | |||
80 | parser.print_help() | 82 | parser.print_help() |
81 | 83 | ||
82 | sys.exit(1) | 84 | sys.exit(1) |
83 | if not os.path.exists(args.buildhistory_dir): | ||
84 | if args.buildhistory_dir == 'buildhistory/': | ||
85 | cwd = os.getcwd() | ||
86 | if os.path.basename(cwd) == 'buildhistory': | ||
87 | args.buildhistory_dir = cwd | ||
88 | 85 | ||
89 | if not os.path.exists(args.buildhistory_dir): | 86 | if not os.path.exists(args.buildhistory_dir): |
90 | sys.stderr.write('Buildhistory directory "%s" does not exist\n\n' % args.buildhistory_dir) | 87 | sys.stderr.write('Buildhistory directory "%s" does not exist\n\n' % args.buildhistory_dir) |