[PATCH 4 of 5 RFC] patchbomb: lay test groundwork for some new functionality

Augie Fackler raf at durin42.com
Mon Jun 19 22:49:26 EDT 2017


> On Jun 19, 2017, at 4:06 AM, Denis Laxalde <denis at laxalde.org> wrote:
> 
> Below is what I tried locally based on patch 5. The --to=users flag has no effect. Am I missing something?
> 
> diff --git a/tests/test-patchbomb.t b/tests/test-patchbomb.t
> --- a/tests/test-patchbomb.t
> +++ b/tests/test-patchbomb.t
> @@ -3191,3 +3191,45 @@ This change modifies file b
>    d
>   +d
> 
> +  $ cat >> .hgemaildefaults <<EOF
> +  > [users]
> +  > cc = mercurial-users at example.com
> +  > EOF
> +  $ hg email --date '1970-1-1 0:2' --to=devel --to=users --test -r 10 --config patchbomb.publicurl=
> +  From [test]: test
> +  this patch series consists of 1 patches.
> +
> +  To [mercurial-devel at example.com]: mercurial-devel at example.com
> +  Cc [c-or-d at example.com mercurial-users at example.com]: c-or-d at example.com mercurial-users at example.com
> +

It had the expected effect: it expanded the alias, which has a Cc: but no To:. That’s (in part) why I didn’t allow aliases to contain @, since it seems extremely unlikely that a mail address without an @ is a user’s intent in the wild (not zero users, but near enough).




More information about the Mercurial-devel mailing list