git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "J. Bruce Fields" <bfields@fieldses.org>
To: Jeff King <peff@peff.net>
Cc: "Karl Hasselström" <kha@treskal.com>, git@vger.kernel.org
Subject: Re: [PATCH 01/10] Add a birdview-on-the-source-code section to the user manual
Date: Tue, 15 May 2007 11:24:58 -0400	[thread overview]
Message-ID: <20070515152457.GC6794@fieldses.org> (raw)
In-Reply-To: <20070515050808.GA11745@coredump.intra.peff.net>

On Tue, May 15, 2007 at 01:08:08AM -0400, Jeff King wrote:
> However, the content-type is already specified, so it shouldn't need to
> rewrite. However, I notice that your original message is missing a
> MIME-Version: 1.0 header. My guess is that vger's logic is that without
> that header, it can't trust the Content-Type you have provided (and
> indeed, not including MIME-Version violates the MIME RFCs, I believe).
> 
> I assumed this was a bug in git-send-email, but looking closer, it
> doesn't put in any mime information at all! So your sending smtp server
> is adding in the content-type header,

Nope...

> but it's failing to add the
> MIME-Version header, which I think is a bug (I can dig up the RFC
> reference if you want).
> 
> Arguably, git should be generating the full MIME header-set, since it
> knows what actual encoding the message is in.

... Yes.  But actually, the Content-Type header is from
git-format-patch:

$ git format-patch --stdout 12806b^..12806b |head
From 12806b65b0d1faec249002c51b871775dc344a47 Mon Sep 17 00:00:00 2001
From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Date: Thu, 10 May 2007 12:36:15 +0200
Subject: [PATCH] Add a birdview-on-the-source-code section to the user
manual
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit

In http://thread.gmane.org/gmane.comp.version-control.git/42479,
a birdview on the source code was requested.

So it's a git-format-patch bug?

--b.

  parent reply	other threads:[~2007-05-15 15:25 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-14 18:19 [PATCH 01/10] Add a birdview-on-the-source-code section to the user manual Karl Hasselström
2007-05-14 18:39 ` J. Bruce Fields
2007-05-14 18:57   ` Matthieu Moy
2007-05-14 18:58   ` Karl Hasselström
     [not found]   ` <20070515042200.GA10884@coredump.intra.peff.net>
2007-05-15  4:50     ` J. Bruce Fields
2007-05-15  5:08       ` Jeff King
2007-05-15  5:57         ` Jeffrey C. Ollie
2007-05-15  6:24           ` Jeff King
2007-05-15  8:24         ` Karl Hasselström
2007-05-15  8:55           ` Junio C Hamano
2007-05-15  9:57             ` Jeff King
2007-05-15 18:41               ` Junio C Hamano
2007-05-16 11:15                 ` Jeff King
2007-05-15 15:24         ` J. Bruce Fields [this message]
2007-05-15 15:35           ` Jeff King
2007-05-15 18:42             ` Junio C Hamano
2007-05-16 11:18               ` Jeff King
  -- strict thread matches above, loose matches on Subject: below --
2007-05-14 15:21 J. Bruce Fields

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=20070515152457.GC6794@fieldses.org \
    --to=bfields@fieldses.org \
    --cc=git@vger.kernel.org \
    --cc=kha@treskal.com \
    --cc=peff@peff.net \
    /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).