[PATCH 2 of 2] templatekw: introduce the changessincelatesttag keyword
yuya at tcha.org
Tue Jun 30 07:21:13 CDT 2015
On Mon, 29 Jun 2015 17:45:27 -0500, Matt Mackall wrote:
> On Mon, 2015-06-29 at 12:45 -0400, Matt Harbison wrote:
> > On Mon, 29 Jun 2015 05:15:20 -0400, Pierre-Yves David
> > <pierre-yves.david at ens-lyon.org> wrote:
> > > rev = maxint
> > > hex = 'f' * 40 (and related node)
> > I assume that means wctx.rev() will return maxint, not just that maxint is
> > magic within a revset? Should we put a wdirrev = None in node for now,
> > and switch it later when ready?
That seems inverse. What we want is an integer other than None for revset.
> > I've got 2 patches that rely on None, and
> > I know I've added others in the past. It seems silly to hunt them all
> > down later, but IDK how soon we are ready to pull the trigger on this
> > change.
> I'm fine with a fair amount of churn here as long as you and Yuya can
> agree on a direction.
My current plan is
wdirrev = 0x7fffffff
repo[None] -> wctx
repo[wdirrev] -> wctx
repo.revs('wdir()') -> [wdirrev]
wctx.rev() -> None
wctx.node() -> None
We might want to change wctx.rev() -> wdirrev and wctx.node() -> 'ff...' at
some time, but it will require a huge amount of careful investigation.
"grep None **/*.py" isn't enough.
More information about the Mercurial-devel