linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: dwmw2@infradead.org (David Woodhouse)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 2/2] Remove now-defunct ts7250 nand driver
Date: Wed, 06 Jan 2010 17:42:17 +0000	[thread overview]
Message-ID: <1262799737.3181.8388.camel@macbook.infradead.org> (raw)
In-Reply-To: <BD79186B4FD85F4B8E60E381CAEE1909020893C9@mi8nycmail19.Mi8.com>

On Wed, 2010-01-06 at 12:26 -0500, H Hartley Sweeten wrote:
> Ok, that's really strange.  I just looked at the email I sent and that line
> is:
> 
> - *   Copyright (C) 2004 Marius Gr?ger (mag at sysgo.de)

Your previous email was in iso8859-1, a legacy charset that really has
no business being used this century. You can see the charset in its
Content-Type: header:
	Content-type: text/plain; charset="iso-8859-1"

The ? was therefore encoded as a single byte 0xf6. When we save the mail
to disk and apply it as a patch, that 0xf6 doesn't actually match what's
in the file you were patching -- and thus the patch didn't apply.

> The character must have been converted as the message left my outbox.

Probably, yes.

> Hmmmm.. What should my default character set be?  It looks like my mailer is
> setup to use these (in this order):

Did you change anything already? Although your older mail was iso8859-1,
the mail to which I'm replying now is different. It's:
	Content-type: text/plain; charset="utf-8"

So this time the ? is encoded as two bytes (0xc3, 0xb6) and _does_ match
what's in the kernel source files. If you'd included the patch in _this_
mail, it would have applied.

> us-ascii
> iso-8859-1
> utf-8 (locale)
> utf-8

It's iso-8859-1 which is causing the problem. Because your mail only
used characters which can be represented in iso8859-1, it got converted
to that. Drop iso8859-1 and it should be fine. ASCII is fine because
it's a subset of UTF-8. Any ASCII text is also valid UTF-8.

-- 
dwmw2

  reply	other threads:[~2010-01-06 17:42 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-05 21:59 [PATCH 2/2] Remove now-defunct ts7250 nand driver H Hartley Sweeten
2010-01-06 13:31 ` David Woodhouse
2010-01-06 17:26   ` H Hartley Sweeten
2010-01-06 17:42     ` David Woodhouse [this message]
2010-01-06 17:47       ` H Hartley Sweeten
2010-05-06 16:47         ` H Hartley Sweeten
2010-05-07  6:01           ` Artem Bityutskiy
2010-05-07 16:37             ` H Hartley Sweeten
2010-01-06 18:07   ` Sending UTF-8 patches (was: [PATCH 2/2] Remove now-defunct ts7250 nand driver) Jamie Lokier
2010-01-06 18:36     ` David Woodhouse
2010-01-06 19:01       ` Nicolas Pitre
2010-01-06 23:21       ` Jamie Lokier
2010-01-06 23:43         ` David Woodhouse
2010-01-06 18:55     ` Nicolas Pitre
2010-01-06 23:05       ` Jamie Lokier
2010-01-06 23:08         ` David Woodhouse
2010-01-06 23:27           ` Jamie Lokier
2010-01-06 23:50             ` David Woodhouse
  -- strict thread matches above, loose matches on Subject: below --
2009-10-06 22:26 [PATCH 2/2] Remove now-defunct ts7250 nand driver H Hartley Sweeten
2009-10-11 11:31 ` Artem Bityutskiy
2009-10-11 21:18   ` H Hartley Sweeten
2009-10-11 21:19   ` H Hartley Sweeten

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=1262799737.3181.8388.camel@macbook.infradead.org \
    --to=dwmw2@infradead.org \
    --cc=linux-arm-kernel@lists.infradead.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).