[PATCH] jcheck: allow bug id line to be ignored

Omair Majid omajid at redhat.com
Mon Apr 8 15:36:16 PDT 2013


Hi Mark,

On 04/08/2013 06:19 PM, mark.reinhold at oracle.com wrote:
> 2013/3/21 5:07 -0700, omajid at redhat.com:
>> The following webrev adds a new configure option "ignore" for bugids. If
>> it is set, then jcheck does not validate the bug summary line at all.
>>
>> http://cr.openjdk.java.net/~omajid/webrevs/jcheck-ignore-bug-line/00/
>>
>> This is needed over at the JDK6 project so we can push commits with
>> Oracle's bug ids (backports from 7 and 8) as well as bugs with
>> "OPENJDK6-XXXX" bug ids. ...
> 
> This is a fine change in principle.  I don't understand, though, why
> you need your first patch (embedded in this patch) to store bugids as
> strings rather than numbers.  You're not going to validate bugids that
> aren't pure numbers, so why bother with that change?

Even thought the bug ids are never used, repo_bugids is called when the
checker object is created:
544     ch = checker(ui, repo, repo_bugids(ui, repo), strict, lax)

And repo_bugids method ends up doing:
209                 b = int(m.group(1))

To find the bug 'number'.

I can avoid this by delaying the invocation of repo_bugids until
necessary, but I went with minimal changes for this patch.

> Please add a case or two to the unit tests to cover this new option.
> The tests for configuration options are at the end of runtests.sh.

Thank you for the pointers; I will post an updated webrev that includes
tests.

Thanks,
Omair

-- 
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681


More information about the hg-tools-dev mailing list