RFR: Make trees work with hg 4.6

Erik Joelsson erik.joelsson at oracle.com
Thu Jun 7 18:46:18 UTC 2018


Hello Martin,

On 2018-06-07 11:25, Martin Buchholz wrote:
> Hej Erik,
>
> You may find my colleague's fixes useful.  I've made them available on cr.
>
> http://cr.openjdk.java.net/~martin/trees/ 
> <http://cr.openjdk.java.net/%7Emartin/trees/>
>
Right, you linked this before, but I never got around to look at it in 
detail. His changes look similar to mine.
> (too many Martins?!)
>
> ---
>
> This change seems unrelated to hg 4.6:
> It seems too easy to get false positives if you're simply looking for 'rc'
> Probably gratuitously different string quotes '' vs "" could be fixed.
>
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.
>
Right, that would make sense. I'm certainly no python expert, I've 
mostly gotten around through pattern patching and googing so far. :)

/Erik
>
> On Mon, Jun 4, 2018 at 3:34 PM, Erik Joelsson 
> <erik.joelsson at oracle.com <mailto:erik.joelsson at oracle.com>> wrote:
>
>     Hello,
>
>     I've rebased my patch for getting trees to work with hg 4.6 on top
>     of Mark's latest commits. This patch also fixes an issue with the
>     internal verlist plugin if the default mercurial on the system
>     running the tests is newer than 3.8. It also widens the filtering
>     of release candidate tags in the hg main repo to catch the 4.6rc1
>     and 4.6rc2 tags.
>
>     Webrev: http://cr.openjdk.java.net/~erikj/trees-4.6/webrev.01/
>     <http://cr.openjdk.java.net/%7Eerikj/trees-4.6/webrev.01/>
>
>     /Erik
>
>



More information about the hg-tools-dev mailing list