Thread (7 messages) flat view 7 messages, 4 authors, 2016-06-15

Re: [PATCH] Checklist clarifications: added "Commits", "Patch:" headings

From: Jari Aalto <hidden>
Date: 2016-06-15 22:43:07

Johannes Schindelin [off-list ref] writes:
Hi,

On Mon, 30 Apr 2007, Jari Aalto wrote:
quoted
+	- send your patch to [off-list ref] by using
+	  git-format-patch(1) with option "-M". See also
+	  git-send-email(1).
 	- do not PGP sign your patch
-	- use "git format-patch -M" to create the patch
NACK.

I never use send-email myself. There are even technical issues to my 
excuse: sometimes I cannot send the mails from my machine. So, this should 
not be a _replacement_ for `git format-patch -M && send mail manually`, 
but only an _alternative_.

However, my impression was that many people had problems when first 
sending patches with send-email. There should be _at least_ a word of 
caution, that you should test the operation on something different than 
the Git list first.
Is this workding okay?

        - send your patch to [off-list ref] by using
          git-format-patch(1) with option "-M". If you use  
          git-send-email(1), please test it first by sending 
          email to yourself.

Jari
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help