[illumos-Developer] [REVIEW] 808 developer tools should support mercurial 1.8

Albert Lee trisk at opensolaris.org
Fri Apr 1 00:27:19 PDT 2011


On Sat, Mar 26, 2011 at 11:25 PM, Garrett D'Amore <garrett at nexenta.com> wrote:
> On Sat, 2011-03-26 at 21:37 -0400, Richard Lowe wrote:
>> webrev:
>>   http://richlowe.net/webrevs/il_808/
>>
>> Yet more (2) conditions as symbols move around, supporting newer
>> mercurial for those that want to use it.
>>
>> -- Rich
>>
>> _______________________________________________
>> Developer mailing list
>> Developer at lists.illumos.org
>> http://lists.illumos.org/m/listinfo/developer
>
>
> Changes look good to me.
>
>        - Garrett
>
>

I was going to suggest hasattr(cmdutil, 'updatedir'), but
Version.at_least() makes more sense when you're also dealing with
changes across 1.6 and 1.4.

LGTM,

-Albert



More information about the Developer mailing list