Re: [PATCH RFC 1/6] send-email: Add --delay for separating emails

From: Nicolas Pitre <nico@cam.org>
Date: Apr 9, 2009, 17:51 UTC
Message-ID: <alpine.LFD.2.00.0904091349150.6741@xanadu.home>
In-Reply-To: <7v3aci43g2.fsf@gitster.siamese.dyndns.org>

On Thu, 9 Apr 2009, Junio C Hamano wrote:

Show 13 quoted lines

> Jeff King <peff@peff.net> writes:
> 
> > I disagree that it's unintrusive (and I understand that you don't think
> > it is, and that there may be others like you, but I am pointing out
> > there are others who think the opposite). But given that it's optional,
> > and it's not very much code, I don't have a strong objection. My
> > original comment was that I was dubious whether it would work: however,
> > I was thinking you would set it to a few seconds. Setting it to 60
> > seconds, I can imagine it would have an impact.
> 
> Heh, then why not make the option specify the number of _minutes_ not
> seconds to delay?  That would help clarifying what this option is meant to
> do.

Maybe only a few seconds is all that is needed in some circumstances. For the issue I just posted, I wouldn't use it at all if the minimum delay was one minute.

Nicolas