From: "David Kågedal" <davidk@lysator.liu.se>
To: git@vger.kernel.org
Subject: Re: [PATCH] Fixed two bugs in git-cvsimport-script.
Date: Wed, 17 Aug 2005 10:07:25 +0200 [thread overview]
Message-ID: <u5tiry50ywy.fsf@lysator.liu.se> (raw)
In-Reply-To: u5tacjjdpxq.fsf@lysator.liu.se
David Kågedal <davidk@lysator.liu.se> writes:
> The git-cvsimport-script had a copule of small bugs that prevented me
> from importing a big CVS repository.
>
> The first was that it didn't handle removed files with a multi-digit
> primary revision number.
I noticed that this patch was accepted, which is great. But there is
a problem with the character encoding in the commit message.
The commit in question is b0921331030d52febf52839753eee1b2b9ca1f24
The "author" field is written as "iso-8859-1?Q?David_K=E5gedal
<davidk@lysator.liu.se>", which is taken from the "From:" line in my
email. This should be decoded by the patch import script before
generating the commit message.
But the trickier question is what encoding to use in the commit
message.
This is the signed-off line in my mail:
Signed-off-by: David Kågedal <davidk@lysator.liu.se>
This is what appears in the commit:
Signed-off-by: David K?5gedal <davidk@lysator.liu.se>
Using ISO-8859-15 or UTF-8 would probably have made more sense here.
Apparently, my mail was encoded as QP, which is not very popular
around here. But it seems that the diff part was decoded properly
before applying. Was that done manually?
Since my name contains a character that is not part of ASCII, it isn't
really an option to send the mails encoded as ASCII. I could probably
convince my mailer (Gnus) to send it as "8bit" or "binary", but that
is a pretty limited solution. An it isn't even legal to use anything
but ASCII in the mail header.
--
David Kågedal
next prev parent reply other threads:[~2005-08-17 8:09 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-08-15 18:18 [PATCH] Fixed two bugs in git-cvsimport-script David Kågedal
2005-08-17 8:07 ` David Kågedal [this message]
2005-08-17 9:22 ` Junio C Hamano
2005-08-17 10:01 ` Junio C Hamano
2005-08-17 11:22 ` David Kågedal
2005-08-17 10:07 ` David Kågedal
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=u5tiry50ywy.fsf@lysator.liu.se \
--to=davidk@lysator.liu.se \
--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