diff options
author | Martin Liska <mliska@suse.cz> | 2021-11-04 14:58:27 +0100 |
---|---|---|
committer | Martin Liska <mliska@suse.cz> | 2021-11-04 15:01:52 +0100 |
commit | af1bfcc04ca1c80af4fc1cda75c6c1d8509c8bd0 (patch) | |
tree | 179e12d7a4efde4eaa10e79e4d5a49a14edc6c5f /contrib/gcc-changelog | |
parent | 5914a7b5c637c9007283226f200dcab8b745abc8 (diff) | |
download | gcc-af1bfcc04ca1c80af4fc1cda75c6c1d8509c8bd0.zip gcc-af1bfcc04ca1c80af4fc1cda75c6c1d8509c8bd0.tar.gz gcc-af1bfcc04ca1c80af4fc1cda75c6c1d8509c8bd0.tar.bz2 |
Add -v option for git_check_commit.py.
Doing so, one can see:
$ git gcc-verify a50914d2111c72d2cd5cb8cf474133f4f85a25f6 -v
Checking a50914d2111c72d2cd5cb8cf474133f4f85a25f6: FAILED
ERR: unchanged file mentioned in a ChangeLog: "gcc/common.opt"
ERR: unchanged file mentioned in a ChangeLog (did you mean "gcc/testsuite/g++.dg/pr102955.C"?): "gcc/testsuite/gcc.dg/pr102955.c"
- gcc/testsuite/gcc.dg/pr102955.c
? ^^ ^
+ gcc/testsuite/g++.dg/pr102955.C
? ^^ ^
contrib/ChangeLog:
* gcc-changelog/git_check_commit.py: Add -v option.
* gcc-changelog/git_commit.py: Print verbose diff for wrong
filename.
Diffstat (limited to 'contrib/gcc-changelog')
-rwxr-xr-x | contrib/gcc-changelog/git_check_commit.py | 4 | ||||
-rwxr-xr-x | contrib/gcc-changelog/git_commit.py | 7 |
2 files changed, 9 insertions, 2 deletions
diff --git a/contrib/gcc-changelog/git_check_commit.py b/contrib/gcc-changelog/git_check_commit.py index 9a4c5d4..d6aff3c 100755 --- a/contrib/gcc-changelog/git_check_commit.py +++ b/contrib/gcc-changelog/git_check_commit.py @@ -29,6 +29,8 @@ parser.add_argument('-g', '--git-path', default='.', help='Path to git repository') parser.add_argument('-p', '--print-changelog', action='store_true', help='Print final changelog entires') +parser.add_argument('-v', '--verbose', action='store_true', + help='Print verbose information') args = parser.parse_args() retval = 0 @@ -41,6 +43,8 @@ for git_commit in parse_git_revisions(args.git_path, args.revisions): else: for error in git_commit.errors: print('ERR: %s' % error) + if args.verbose and error.details: + print(error.details) retval = 1 exit(retval) diff --git a/contrib/gcc-changelog/git_commit.py b/contrib/gcc-changelog/git_commit.py index 60377b6..27a1d59 100755 --- a/contrib/gcc-changelog/git_commit.py +++ b/contrib/gcc-changelog/git_commit.py @@ -197,9 +197,10 @@ def decode_path(path): class Error: - def __init__(self, message, line=None): + def __init__(self, message, line=None, details=None): self.message = message self.line = line + self.details = details def __repr__(self): s = self.message @@ -687,9 +688,11 @@ class GitCommit: for file in sorted(mentioned_files - changed_files): msg = 'unchanged file mentioned in a ChangeLog' candidates = difflib.get_close_matches(file, changed_files, 1) + details = None if candidates: msg += f' (did you mean "{candidates[0]}"?)' - self.errors.append(Error(msg, file)) + details = '\n'.join(difflib.Differ().compare([file], [candidates[0]])).rstrip() + self.errors.append(Error(msg, file, details)) for file in sorted(changed_files - mentioned_files): if not self.in_ignored_location(file): if file in self.new_files: |