Re: [PATCH] doc: move rev-list option -n from git-log.txt to rev-list-options.txt

2012-09-08 Thread Nguyen Thai Ngoc Duy
On Sat, Sep 8, 2012 at 12:14 AM, Junio C Hamano gits...@pobox.com wrote:
 Michael J Gruber g...@drmicha.warpmail.net writes:
  Documentation/git-log.txt  | 6 ++
  Documentation/rev-list-options.txt | 3 ++-
  2 tập tin đã bị thay đổi, 4 được thêm vào(+), 5 bị xóa(-)

 That is one reason why core.local=C (repo specific) and git -c
 core.locale=C (can be used in an alias) would be useful ;)

 Or LC_ALL=C LANG=C git format-patch 

The only problem is I forget to do that from time to time (and doing
that bothers me too)

 It does not bother me (even though I do not read Vietnamese), but
 this has been brought up a few times, and we may want to revert the
 i18n of the diffstat summary.  It does not seem to add much value to
 the system but annoys people.

That's one step towards a better interface for non English speaking
users. git log interface for example still shows Author, Commit,
Date in English and these strings are shared with format-patch.
Reverting back to English to me is a step back.

This brings back to a series I posted about two weeks ago and got no comments

http://article.gmane.org/gmane.comp.version-control.git/204285

I think it's a reasonable approach. Use English for machine interface,
otherwise a native language if available.

 After all, the upstream diffstat
 does not localizes this string (I just checked diffstat-1.55 with
 Jan 2012 timestamp).
-- 
Duy
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] doc: move rev-list option -n from git-log.txt to rev-list-options.txt

2012-09-07 Thread Michael J Gruber
Nguyễn Thái Ngọc Duy venit, vidit, dixit 06.09.2012 16:28:
 rev-list-options.txt is included in git-rev-list.txt. This makes sure
 rev-list man page also shows that, and at one place, together with
 equivalent options -n and --max-count.
 
 Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
 ---
  Documentation/git-log.txt  | 6 ++
  Documentation/rev-list-options.txt | 3 ++-
  2 tập tin đã bị thay đổi, 4 được thêm vào(+), 5 bị xóa(-)

That is one reason why core.local=C (repo specific) and git -c
core.locale=C (can be used in an alias) would be useful ;)

 
 diff --git a/Documentation/git-log.txt b/Documentation/git-log.txt
 index 1f90620..585dac4 100644
 --- a/Documentation/git-log.txt
 +++ b/Documentation/git-log.txt
 @@ -24,10 +24,6 @@ each commit introduces are shown.
  OPTIONS
  ---
  
 --n::
 - Limits the number of commits to show.
 - Note that this is a commit limiting option, see below.
 -
  since..until::
   Show only commits between the named two commits.  When
   either since or until is omitted, it defaults to
 @@ -137,6 +133,8 @@ Examples
   This makes sense only when following a strict policy of merging all
   topic branches when staying on a single integration branch.
  
 +`git log -3`::
 + Limits the number of commits to show to 3.
  
  Discussion
  --
 diff --git a/Documentation/rev-list-options.txt 
 b/Documentation/rev-list-options.txt
 index def1340..1b15ea9 100644
 --- a/Documentation/rev-list-options.txt
 +++ b/Documentation/rev-list-options.txt
 @@ -8,7 +8,8 @@ ordering and formatting options, such as '--reverse'.
  
  --
  
 --n 'number'::
 +-number::
 +-n number::
  --max-count=number::
  
   Limit the number of commits to output.
 

This looks OK.

I noticed though that the man pages of git-log and git-rev-list still
look more different than they would need to, e.g. regarding the way
limitting by paths is explained more prominently with git-log. But that
may just be being more user friendly for git-log's man page than for git
rev-list's, which is OK.

Michael
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] doc: move rev-list option -n from git-log.txt to rev-list-options.txt

2012-09-07 Thread Junio C Hamano
Michael J Gruber g...@drmicha.warpmail.net writes:

 Nguyễn Thái Ngọc Duy venit, vidit, dixit 06.09.2012 16:28:
 rev-list-options.txt is included in git-rev-list.txt. This makes sure
 rev-list man page also shows that, and at one place, together with
 equivalent options -n and --max-count.
 
 Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
 ---
  Documentation/git-log.txt  | 6 ++
  Documentation/rev-list-options.txt | 3 ++-
  2 tập tin đã bị thay đổi, 4 được thêm vào(+), 5 bị xóa(-)

 That is one reason why core.local=C (repo specific) and git -c
 core.locale=C (can be used in an alias) would be useful ;)

Or LC_ALL=C LANG=C git format-patch 

It does not bother me (even though I do not read Vietnamese), but
this has been brought up a few times, and we may want to revert the
i18n of the diffstat summary.  It does not seem to add much value to
the system but annoys people.  After all, the upstream diffstat
does not localizes this string (I just checked diffstat-1.55 with
Jan 2012 timestamp).
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html