[PATCH 12 of 12 RFC] policy: add helper to import cext/pure module
Jun Wu
quark at fb.com
Tue May 9 14:08:03 EDT 2017
Excerpts from Yuya Nishihara's message of 2017-05-09 23:09:07 +0900:
> The auto-rebuild could be HGMODULEPOLICY=compile if we want.
I actually prefer "allow" by default to auto build. People wanting
up-to-date binaries should run `make local` manually.
> > Patch 12 is where I disagree. _dualmod still requires careful renaming and
> > could cause subtle problems that does not get warned by any code checking
> > tools. They also require extra effort in writing and reviewing related code.
> > I'd like to go through the explicit versioning way so even the renaming
> > churn could be avoided.
> >
> > [1]: https://www.mercurial-scm.org/pipermail/mercurial-devel/2017-May/097329.html
>
> I'm okay for the explicit versioning. I can adjust the last patch for that.
>
> I don't have strong preference since managing API versions would be as
> simple as managing explicit version constants unless we have internal
> dependency in C layer.
We do have complex cases: module-level exception (mpatch.mpatchError), and
parsers.c with complex Python objects. Think about if someone wants to catch
mpatchError, or change the return value of lmiter_iterkeysnext in manifest.c
for example.
More information about the Mercurial-devel
mailing list