git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andy Parkins <andyparkins@gmail.com>
To: git@vger.kernel.org
Subject: git-imap-send and "From "
Date: Sat, 21 Oct 2006 17:51:17 +0100	[thread overview]
Message-ID: <200610211751.21581.andyparkins@gmail.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 1804 bytes --]

Hello,

I've been playing around with git; made some simple patches for git and 
thought I would forward them to the mailing list.  I did this using

git-format-patch --stdout --signoff -M -B | git-imap-send

With the following in .gitconfig and courier-imap as the IMAP server.

[imap]
  Folder = "INBOX.Drafts"
  Tunnel = "ssh -q mailhost /usr/bin/imapd ./Maildir 2> /dev/null"

This appeared to work as expected.  A nicely formatted patch appeared in my 
drafts mailbox for checking and sending; which I did.  However, those emails 
caused a few problems; like:

"Can you please not send your emails to "unlisted recipients?"  It
breaks my mail filtering and your mails don't end up going to my "git"
folder."

As far as I knew, I hadn't done this.  The email looked fine when I checked it 
in KMail.  So, I went back and had a look at the raw source of the patch 
email that I'd sent and found this:

From 0e3c0aefc3276bd271553d171ed9bcc52d85230e Mon Sep 17 00:00:00 2001
From: Andy Parkins <andyparkins@gmail.com>
Date: Fri, 20 Oct 2006 15:24:40 +0100
Subject: [PATCH] Use email address only for looking up signing key in git-tag

It's that first line that's the problem.  It was generated by git-format-patch 
of course, however, I assume that it was intended to be stripped on the way 
to the IMAP server.  So,

 * Courier's /usr/bin/imapd should have stripped it and didn't
 * git-imap-send shouldn't have sent it

Can anyone help me out with which it should be?  Or point me at the relevant 
RFC?

While I was poking around I found that git is hard coded with

printf("From %s Mon Sep 17 00:00:00 2001\n", sha1);

Was it intentional that the date be hard coded like that?


Andy
-- 
Dr Andrew Parkins, M Eng (Hons), AMIEE
andyparkins@gmail.com

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

             reply	other threads:[~2006-10-21 16:55 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-21 16:51 Andy Parkins [this message]
2006-10-21 17:55 ` git-imap-send and "From " Jeff King
2006-10-21 18:01   ` Jeff King
2006-10-21 18:11   ` Andy Parkins

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200610211751.21581.andyparkins@gmail.com \
    --to=andyparkins@gmail.com \
    --cc=git@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).