[PATCH 2 of 2] notify: support revset selection for subscriptions
dgc at uchicago.edu
Wed Jun 6 22:04:50 CDT 2012
* On 06 Jun 2012, Matt Mackall wrote:
> On Tue, 2012-05-29 at 07:58 -0500, David Champion wrote:
> > # HG changeset patch
> > # User David Champion <dgc at uchicago.edu>
> > # Date 1338296186 18000
> > # Node ID b63788f02e53f1ecf00281adc71f19656e11dcd1
> > # Parent 222934f53810943a6e7eb74ebd756ea8cafa810e
> > notify: support revset selection for subscriptions
> This really wants to use repo.revs():
> The syntax of the expression is described here:
> It can probably be as simple as:
> revs = repo.revs("%r and %ld", filterspec, listofnewrevs)
I'm new to revsets, and didn't know repo.revs(), but I thought that
limiting the scope of search in python rather than in revset grammer
would be faster. I'm guessing that it probably does, but not enough to
matter in a single-pass setup utility, so it's better to use a familiar
> > A repo pattern for any notify configuration contains a glob matching the
> > path to the repo. Additionally, it may now contain a revset spec, separated
> > from the glob by '//'.
> That separator is a bit unprecedented around here. There is however some
> precedent for '#'.
I'm fine with that, but I picked '//' because (in POSIX pathspace
anyway) it's unlikely to be otherwise meaningful: it's equivalent to
'/', and it doesn't use up a character that could appear in filenames.
Just making the note -- I'll switch to '#' if that's still your
> > Example:
> > [reposubs]
> > */widgets//branch(release) qa-team at example.com
> That syntax looks weird.. missing '='?
Oops, thanks. I'll update and resend patch 2.
David Champion • dgc at uchicago.edu • IT Services • University of Chicago
More information about the Mercurial-devel