[PATCH 5 of 5 RFC] patchbomb: add support for a file that configures default To and CC

Yuya Nishihara yuya at tcha.org
Wed Jun 14 10:17:43 EDT 2017


On Tue, 13 Jun 2017 18:50:27 -0400, Augie Fackler wrote:
> Durham had the idea of a tweak to patchbomb that would let us not do https://www.mercurial-scm.org/wiki/Herald.
> 
> I implemented filesets for the RFC, but I think we should probably also allow reset-based subscriptions too (e.g. "histedit" as a keyword or similar). I'd appreciate any feedback people have, since this was a quick project and it feels like it might have some real utility.

Looks pretty nice.

> > @@ -3036,12 +3036,17 @@ Test hg email defaults settings:
> >> [devel.setsubscriptions]
> >> c or d = c-or-d at example.com
> >> a or b = a-or-b at example.com
> > +  > [foo.setsubscriptions]
> > +  > binary() = hates-binary-files at example.com

Perhaps the config syntax could be arbitrary_key:attr = value.

  hatebin:fileset = binary()
  hatebin:revset = keyword(binary)
  hatebin:cc = hates-binary-files at example.com

fileset and revset can be quite long. We wouldn't want to write them as config
keys.

And I prefer using matcher patterns instead of fileset for consistency with
the other commands.


More information about the Mercurial-devel mailing list