[Patch, v2] contrib/mklog.py: Improve PR handling (was: Re: git gcc-commit-mklog doesn't extract PR number to ChangeLog)
Mon Jun 21 08:37:37 GMT 2021
On 21.06.21 10:09, Martin Liška wrote:
> $ pytest test_mklog.py
> FAILED test_mklog.py::TestMklog::test_sorting - AssertionError: assert
> '\n\tPR 50209...New test.\n\n' == 'gcc/ChangeLo...New test.\n\n'
Aha, missed that there is indeed a testsuite - nice!
> $ flake8 mklog.py
> mklog.py:187:23: Q000 Remove bad quotes
I have now filled:
>> + # PR number in the file name
>> + fname = os.path.basename(file.path)
> This is a dead code.
>> + fname = os.path.splitext(fname)
>> + m = pr_filename_regex.search(fname)
It does not look like dead code to me.
>> + parser.add_argument('-b', '--pr-numbers', action='append',
>> + help='Add the specified PRs (comma separated)')
> Do we really want to support '-b 1 -b 2' and also -b '1,2' formats?
> Seems to me quite
I don't have a strong opinion. I started with '-b 123,245', believing
that the syntax is fine. But then I realized that without '-p'
specifying multiple '-b' looks better by having multiple '-b' if 'PR
<component>/' (needed for -p as the string is than taken as is). Thus,
I ended up supporting either variant.
But I also happily drop the ',' support.
Change: One quote change, one test_mklog update.
Mentor Graphics (Deutschland) GmbH, Arnulfstrasse 201, 80634 München Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Frank Thürauf
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 5950 bytes
Desc: not available
More information about the Gcc-patches