[PATCH] revset: added smartset attribute to new classes to test at mfunc and getset

Pierre-Yves David pierre-yves.david at ens-lyon.org
Tue Feb 18 18:40:58 CST 2014



On 02/18/2014 04:37 PM, Lucas Moscovicz wrote:
>
>
> On 2/18/14, 4:25 PM, "Pierre-Yves David" <pierre-yves.david at ens-lyon.org>
> wrote:
>
>>
>>
>> On 02/18/2014 04:13 PM, Lucas Moscovicz wrote:
>>> # HG changeset patch
>>> # User Lucas Moscovicz <lmoscovicz at fb.com>
>>> # Date 1392767686 28800
>>> #      Tue Feb 18 15:54:46 2014 -0800
>>> # Node ID d9813f85f80e36e413da7bd1b5edd3a2118bc715
>>> # Parent  c29948fed40a2d9755ecaa01ec05bfa542f65670
>>> revset: added smartset attribute to new classes to test at mfunc and
>>> getset
>>
>> I believe this patch is important.  There is a lots of extension out
>> there implementing new revset. We are adding smarter feature which are
>> mostly a super set of the old behavior (using list). There is no good
>> reason to break the compatibility for our user out there.
>>
>> I know we do not have a public API but this does not mean we should
>> gratuitously break widely used a trivial API. (I could compare that to
>> breaking the changectx API for no good reason).
>>
>>> Now extensions shouldn't break when adding new revsets.
>>
>> Did you actually test it with old style code?
>
> I just tested it with an extension that used to fail over this same
> problem and I had fixed manually.

Which extension? (out of curiosity)

-- 
Pierre-Yves


More information about the Mercurial-devel mailing list