{"thread":{"id":"23372","subject":"[PATCH] Documentation/SubmittingPatches: Clarify Gmail section","startedAt":"2010-04-07T17:52:28Z","lastAt":"2010-04-07T20:00:47Z","messageCount":4,"participants":["Ramkumar Ramachandra","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"138862","messageId":"s2kf3271551004071052o58656534z80de5068b207bdb2@mail.gmail.com","threadId":"23372","inReplyTo":null,"subject":"[PATCH] Documentation/SubmittingPatches: Clarify Gmail section","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2010-04-07T17:52:28Z","receivedAt":"2010-04-07T17:52:28Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Clarify the Gmail section: the user should not touch the message body\nafter using \"git imap-send\", otherwise the patch will get mangled.\n\nSigned-off-by: Ramkumar Ramachandra \n---\n Documentation/SubmittingPatches | 36 ++++++++++++++++++------------------\n 1 files changed, 18 insertions(+), 18 deletions(-)\n\ndiff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches\nindex c686f86..ff0e1b6 100644\n--- a/Documentation/SubmittingPatches\n+++ b/Documentation/SubmittingPatches\n@@ -517,15 +517,20 @@ message, complete the addressing and subject\nfields, and press send.\n Gmail\n -----\n\n-GMail does not appear to have any way to turn off line wrapping in the web\n-interface, so this will mangle any emails that you send. You can however\n-use any IMAP email client to connect to the google imap server, and forward\n-the emails through that. Just make sure to disable line wrapping in that\n-email client. Alternatively, use \"git send-email\" instead.\n+GMail does not have any way to turn off line wrapping in the web\n+interface, so this will mangle any emails that you send. You have a\n+couple of options here:\n\n-Submitting properly formatted patches via Gmail is simple now that\n-IMAP support is available. First, edit your ~/.gitconfig to specify your\n-account settings:\n+1. Use an email client that doesn't automatically wrap lines to send\n+emails through the Google SMTP server, or use \"git send-email\" to\n+achieve the same effect.\n+\n+2. Use \"git imap-send\" to push an email to the GMail Drafts folder via\n+IMAP. Then use the GMail web interface to fill out the To: and CC:\n+fields, taking care not to touch the message body, and send away!\n+\n+To get it working, edit your ~/.gitconfig to specify your account\n+settings:\n\n [imap]\n \tfolder = \"[Gmail]/Drafts\"\n@@ -536,16 +541,11 @@ account settings:\n \tsslverify = false\n\n You might need to instead use: folder = \"[Google Mail]/Drafts\" if you\nget an error\n-that the \"Folder doesn't exist\".\n-\n-Next, ensure that your Gmail settings are correct. In \"Settings\" the\n-\"Use Unicode (UTF-8) encoding for outgoing messages\" should be checked.\n+that the \"Folder doesn't exist\". Also ensure that under the GMail\n+\"Settings\", the \"Use Unicode (UTF-8) encoding for outgoing messages\"\n+is checked.\n\n-Once your commits are ready to send to the mailing list, run the following\n-command to send the patch emails to your Gmail Drafts folder.\n+Once your commits are ready to send to the mailing list, run the\n+following command.\n\n \t$ git format-patch -M --stdout origin/master | git imap-send\n-\n-Go to your Gmail account, open the Drafts folder, find the patch email, fill\n-in the To: and CC: fields and send away!\n-\n-- \n1.7.0.3\n"},{"id":"138865","messageId":"t2of3271551004071108ic4393a8v3ae9929f01b8c086@mail.gmail.com","threadId":"23372","inReplyTo":"s2kf3271551004071052o58656534z80de5068b207bdb2@mail.gmail.com","subject":"Re: [PATCH] Documentation/SubmittingPatches: Clarify Gmail section","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2010-04-07T18:08:49Z","receivedAt":"2010-04-07T18:08:49Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Hi,\n\nI can't be sure that I'm right about this. Could someone please confirm?\nIronically, this patch seems to be broken. I think it has something to\ndo with the auto-save (for drafts) in the web interface.\n\n-- Ram\n"},{"id":"138878","messageId":"7v8w8zcc5h.fsf@alter.siamese.dyndns.org","threadId":"23372","inReplyTo":"t2of3271551004071108ic4393a8v3ae9929f01b8c086@mail.gmail.com","subject":"Re: [PATCH] Documentation/SubmittingPatches: Clarify Gmail section","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2010-04-07T19:09:46Z","receivedAt":"2010-04-07T19:09:46Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ramkumar Ramachandra writes:\n\n> I can't be sure that I'm right about this. Could someone please confirm?\n> Ironically, this patch seems to be broken. I think it has something to\n> do with the auto-save (for drafts) in the web interface.\n\nDidn't I already say that it is _known_ not to work?\n\nAs you earlier said that the documentation of imap-send states GMail web\nworks, perhaps that description is incorrect and needs fixing.\n\nSee the thread that begins with this message in the archive:\n\n Message-ID: <1265555642-40204-1-git-send-email-git@aaroncrane.co.uk>\n From: Aaron Crane \n Subject: [PATCH] Documentation/SubmittingPatches: fix Gmail workaround advice\n Date: Sun, 7 Feb 2010 15:14:02 +0000\n\ne.g.\n\n http://thread.gmane.org/gmane.comp.version-control.git/139240\n"},{"id":"138886","messageId":"n2tf3271551004071300pe6ae8f42v8f0a19e1b6cec272@mail.gmail.com","threadId":"23372","inReplyTo":"7v8w8zcc5h.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Documentation/SubmittingPatches: Clarify Gmail section","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2010-04-07T20:00:47Z","receivedAt":"2010-04-07T20:00:47Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"> Didn't I already say that it is _known_ not to work?\n\nRight. Sorry about that. While testing, I must have made the mistake\nof using a patch that didn't not have long enough lines to be\nlinewrapped.\n\n> As you earlier said that the documentation of imap-send states GMail web\n> works, perhaps that description is incorrect and needs fixing.\n\nThanks for the pointer. I've prepared another revision.\n\n-- Ram\n"}]}