From: Eric Wong <normalperson@yhbt.net>
To: Robert Pollak <robert.pollak@gmail.com>
Cc: Dmitry Statyvka <dstatyvka@tmsoft-ltd.kiev.ua>, git@vger.kernel.org
Subject: Re: [PATCH] git svn: add an option to recode pathnames
Date: Fri, 30 Jul 2010 07:59:21 +0000 [thread overview]
Message-ID: <20100730075921.GA9596@dcvr.yhbt.net> (raw)
In-Reply-To: <4C523935.5070504@gmail.com>
Robert Pollak <robert.pollak@gmail.com> wrote:
> Introduce a new option 'svn.pathnameencoding' that instructs git svn to
> recode pathnames to a given encoding. It can be used by windows users
> and by those who work in non-utf8 locales to avoid corrupted file names
> with non-ascii characters.
>
> Signed-off-by: Dmitry Statyvka <dstatyvka@tmsoft-ltd.kiev.ua>
> [robert.pollak@gmail.com: renamed the option and added manpage documentation]
> Signed-off-by: Robert Pollak <robert.pollak@gmail.com>
> ---
>
> Hello Eric,
>
> since the patch is useful to me, I have made the requested option name
> change and added some manpage documentation. Please consider applying the
> patch or give me additional feedback.
Thanks Robert!
It looks alright to me and I've acked and pushed it out to
git://git.bogomips.org/git-svn (crediting Dmitry as the author)
If it's not too much trouble, having a test case to ensure it stays
working with future changes would be nice.
> This is the first patch I submit on this list, so I hope it's ok.
For addendum changes, the standard we seem to have adopted is
first initials above the Signed-off/Acked-by lines:
>From 3713e2226bcda64513efd537f370ce4d7f767a1e Mon Sep 17 00:00:00 2001
From: Dmitry Statyvka <dstatyvka@tmsoft-ltd.kiev.ua>
Date: Fri, 30 Jul 2010 04:30:13 +0200
Subject: [PATCH] git svn: add an option to recode pathnames
Introduce a new option 'svn.pathnameencoding' that instructs git svn to
recode pathnames to a given encoding. It can be used by windows users
and by those who work in non-utf8 locales to avoid corrupted file names
with non-ascii characters.
[rp: renamed the option and added manpage documentation]
Signed-off-by: Dmitry Statyvka <dstatyvka@tmsoft-ltd.kiev.ua>
Signed-off-by: Robert Pollak <robert.pollak@gmail.com>
Acked-by: Eric Wong <normalperson@yhbt.net>
---
<snip>
--
Eric Wong
next prev parent reply other threads:[~2010-07-30 7:59 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-29 9:16 [PATCH] git svn: add an option to recode pathnames Dmitry Statyvka
2009-07-29 18:59 ` Eric Wong
2010-07-30 2:30 ` Robert Pollak
2010-07-30 7:59 ` Eric Wong [this message]
2010-08-02 10:36 ` Dmitry Statyvka
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=20100730075921.GA9596@dcvr.yhbt.net \
--to=normalperson@yhbt.net \
--cc=dstatyvka@tmsoft-ltd.kiev.ua \
--cc=git@vger.kernel.org \
--cc=robert.pollak@gmail.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.