[PATCH 2 of 9] mercurial: add an extension module for fast dict operations

Matt Mackall mpm at selenic.com
Thu Mar 28 11:54:28 CDT 2013


On Mon, 2013-03-25 at 10:22 -0700, Bryan O'Sullivan wrote:
> On Sun, Mar 24, 2013 at 5:27 PM, Siddharth Agarwal <sid0 at fb.com>
> wrote:
>         +    Extension('mercurial.dicthelpers',
>         ['mercurial/dicthelpers.c']),
>         
> 
> I have a hazy recollection of a conversation with Matt a few years
> ago, from which external packagers have some difficulty with new
> extension modules. I have no details, but that's why parsers.c
> contains a bunch of random very-much-not-parsers code.

New C APIs tend to break our usual running-is-compiling development
model.

That's pain for us (when updating, when bisecting, etc.) and pain for
users who are upgrading by checkout. Some of the latter don't even have
compilers (the sort of people using HackableMercurial).

There may be occasional packaging glitches too. THG has failed to
package newly-added files on several releases, not really sure why this
happens.

-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list