* [BUG] git-send-email format-patch awareness & retry
@ 2010-09-07 16:59 Ævar Arnfjörð Bjarmason
2010-09-08 9:14 ` Thomas Rast
0 siblings, 1 reply; 4+ messages in thread
From: Ævar Arnfjörð Bjarmason @ 2010-09-07 16:59 UTC (permalink / raw)
To: Git Mailing List; +Cc: Junio C Hamano, Ryan Anderson, Greg KH
(cc primary authors & Junio)
This is more of a TODO note to myself than a bug, but if someone else
gets around to fixing these that'd be great too.
git-send-email should:
* Complain loudly on "*** SUBJECT HERE ***" and "*** BLURB HERE ***"
This should have an additional "are you sure" dialog:
Subject: [PATCH 00/20] *** SUBJECT HERE ***
Date: Tue, 7 Sep 2010 16:43:03 +0000
Message-Id: <1283877803-17757-1-git-send-email-avarab@gmail.com>
X-Mailer: git-send-email 1.7.2.3.313.gcd15
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Send this email? ([y]es|[n]o|[q]uit|[a]ll): y
Password:
* Retry before bailing out:
It's annoying to lose state because of temporary SMTP errors. This
should offer to sleep for a few seconds and retry:
Content-Transfer-Encoding: 8bit
Send this email? ([y]es|[n]o|[q]uit|[a]ll): y
Go ahead k83sm4304468weq.14
4.3.0 Mail server temporarily rejected message. k83sm4304468weq.14
It would also be a lot easier to catch errors in the git-send-email
output if it supported colors, e.g. colored the subject line
differently from others.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [BUG] git-send-email format-patch awareness & retry
2010-09-07 16:59 [BUG] git-send-email format-patch awareness & retry Ævar Arnfjörð Bjarmason
@ 2010-09-08 9:14 ` Thomas Rast
2010-09-08 16:04 ` Junio C Hamano
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Rast @ 2010-09-08 9:14 UTC (permalink / raw)
To: Ævar Arnfjörð Bjarmason
Cc: Git Mailing List, Junio C Hamano, Ryan Anderson, Greg KH
Ævar Arnfjörð Bjarmason wrote:
> (cc primary authors & Junio)
>
> This is more of a TODO note to myself than a bug, but if someone else
> gets around to fixing these that'd be great too.
>
> git-send-email should:
>
> * Complain loudly on "*** SUBJECT HERE ***" and "*** BLURB HERE ***"
>
> This should have an additional "are you sure" dialog:
>
> Subject: [PATCH 00/20] *** SUBJECT HERE ***
> Date: Tue, 7 Sep 2010 16:43:03 +0000
> Message-Id: <1283877803-17757-1-git-send-email-avarab@gmail.com>
> X-Mailer: git-send-email 1.7.2.3.313.gcd15
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> Send this email? ([y]es|[n]o|[q]uit|[a]ll): y
> Password:
You could resurrect
http://mid.gmane.org/7fedc4b76fed03e4db6a2bb7453609c9cd02928a.1244496564.git.trast@student.ethz.ch
though on re-reading I'm not so sure anymore that *any* complaints
remained.
--
Thomas Rast
trast@{inf,student}.ethz.ch
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [BUG] git-send-email format-patch awareness & retry
2010-09-08 9:14 ` Thomas Rast
@ 2010-09-08 16:04 ` Junio C Hamano
2010-09-09 8:08 ` Thomas Rast
0 siblings, 1 reply; 4+ messages in thread
From: Junio C Hamano @ 2010-09-08 16:04 UTC (permalink / raw)
To: Thomas Rast
Cc: Ævar Arnfjörð Bjarmason, Git Mailing List,
Ryan Anderson, Greg KH
Thomas Rast <trast@student.ethz.ch> writes:
> You could resurrect
>
> http://mid.gmane.org/7fedc4b76fed03e4db6a2bb7453609c9cd02928a.1244496564.git.trast@student.ethz.ch
>
> though on re-reading I'm not so sure anymore that *any* complaints
> remained.
I think I got distracted in the follow-up discussion and forgot about that
patch.
And the patch looks sane; let's queue it, so that I won't forget about it
again.
Thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [BUG] git-send-email format-patch awareness & retry
2010-09-08 16:04 ` Junio C Hamano
@ 2010-09-09 8:08 ` Thomas Rast
0 siblings, 0 replies; 4+ messages in thread
From: Thomas Rast @ 2010-09-09 8:08 UTC (permalink / raw)
To: Junio C Hamano
Cc: Ævar Arnfjörð Bjarmason, Git Mailing List,
Ryan Anderson, Greg KH
Junio C Hamano wrote:
> I think I got distracted in the follow-up discussion and forgot about that
> patch.
>
> And the patch looks sane; let's queue it, so that I won't forget about it
> again.
>
> Thanks.
Thanks. No need to apologize; I forgot too, and only remembered when
Ævar mentioned the topic again.
--
Thomas Rast
trast@{inf,student}.ethz.ch
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-09-09 8:09 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-09-07 16:59 [BUG] git-send-email format-patch awareness & retry Ævar Arnfjörð Bjarmason
2010-09-08 9:14 ` Thomas Rast
2010-09-08 16:04 ` Junio C Hamano
2010-09-09 8:08 ` Thomas Rast
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.