Re: [PATCH] send-email: error out when relogin delay is missing
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Feb 8, 2018, 08:08 UTC
- Message-ID
- <CAPig+cQejz+1ZWep31hw0pjxUoewRt+-2WATtw6PEusfvhn8EA@mail.gmail.com>
- In-Reply-To
- <20180207234359.31493-1-sbeller@google.com>
On Wed, Feb 7, 2018 at 6:43 PM, Stefan Beller <sbeller@google.com> wrote:
Show 13 quoted lines
> [...]
> Error out for now instead of potentially confusing the user.
> As 5453b83bdf (send-email: --batch-size to work around some SMTP
> server limit, 2017-05-21) lays out, we rather want to not have this
> interface anyway and would rather want to react on the server throttling
> dynamically.
>
> Signed-off-by: Stefan Beller <sbeller@google.com>
> ---
> diff --git a/git-send-email.perl b/git-send-email.perl
> @@ -379,6 +379,9 @@ unless ($rc) {
> +die __("When a batch size is given, the relogin delay must be set\n")
> + if defined $relogin_delay and not defined $batch_size;This only makes sense is 'batch-size' is specified but not 'relogin'. If the other way around, then the error is confusing. How about this instead?
"--batch-size and --relogin must be specified together"
...or something.