RFR: Make trees work with hg 4.6

Martin Buchholz martinrb at google.com
Wed Jul 11 15:09:15 UTC 2018


Can we get the pending changes to hg tools committed please?

Would it help if I reviewed them?

Would it help if code tools had more reviewers?  Perhaps deputize regular
openjdk reviewers?

On Fri, Jun 8, 2018 at 11:46 AM, Erik Joelsson <erik.joelsson at oracle.com>
wrote:

> Hello,
> On 2018-06-07 11:59, Martin Buchholz wrote:
>
>
>
> On Thu, Jun 7, 2018 at 11:46 AM, Erik Joelsson <erik.joelsson at oracle.com>
> wrote:
>
>> This is because the last two release candidate tags in the official hg
>> repo are "4.6rc1" and "4.6rc2". They changed the pattern. I don't see "rc"
>> anywhere else in tags that shouldn't be matched. This plugin is only used
>> internally in trees testing.
>>
>> -    tags = filter(lambda t: not t.endswith('-rc') and t != "tip", tags)
>> +    tags = filter(lambda t: 'rc' not in t and t != "tip", tags)
>>
>> The other changes look good to me but I'm not a python or trees developer.
>>
>>
> Ohh.... verlist is being used to get tags for the upstream mercurial repo
> for mercurial itself?  If so, maybe document that?
>
> I think the comment at the top basically says this.
>
> Looking for the substring "rc" seems particularly brittle because it
> happens to match "mercurial".
>
> Well, we can only really make this code good enough to match the current
> tags in the official mercurial repo. If they change format again, we will
> need to adapt. I've changed the rc and tip strings to all use single quote
> on that line.
>
> /Erik
>


More information about the hg-tools-dev mailing list