Aw: Re: [PATCH v3 1/6] Change old system name 'GIT' to 'Git'

2013-01-22 Thread Thomas Ackermann
 
 I think it misses GIT - the stupid content tracker in README, but
 probably it is OK (it is not an end-user facing documentation).
 
I only checked ./Documentation; but this should be changed also.

 I noticed that these two places still use poor-man's small caps
 after this patch.
 
Thanks.

  * Documentation/SubmittingPatches:
  that are being emailed around. Although core GIT is a lot

I will change this.
 
  * Documentation/git-credential.txt:
  TYPICAL USE OF GIT CREDENTIAL

This is to be debated because here all caps is used as poor man's bold face
and it would look odd if we write 'TYPICAL USE OF Git CREDENTIAL'?
 
 
 Not commenting on other 5 patches in the series yet, but if they
 interact with other topics in flight, they may have to be separated
 out. We'll see.
 
Perhaps it would be best to separate this into 2 series:
[1/6]+[6/6] and [2/6]-[5/6]?



---
Thomas
--
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: Aw: Re: [PATCH v3 1/6] Change old system name 'GIT' to 'Git'

2013-01-22 Thread Junio C Hamano
Thomas Ackermann th.ac...@arcor.de writes:

  * Documentation/git-credential.txt:
  TYPICAL USE OF GIT CREDENTIAL

 This is to be debated because here all caps is used as poor man's bold face
 and it would look odd if we write 'TYPICAL USE OF Git CREDENTIAL'?

I left this as-is in my fixup commit, as the html and manpage
backends seem to typeset the section titles differently.  I however
have to wonder what the result will look like if we replaced it with

Typical use of Git credential

Can you try and see how well it works?

 Perhaps it would be best to separate this into 2 series:
 [1/6]+[6/6] and [2/6]-[5/6]?

I'd really hate to see these patches posted again to the list and
asking for re-reviewing the whole thing.  Can we go incremental
while parking these original patches on 'pu', and later squash the
whole thing into two commits, or something?

Right now I have these on 'pu':

  b15ac19 Add rule for when to use 'git' and when to use 'Git'
  57d4b2c Change 'git' to 'Git' whenever the whole system is referred to #4
  01758e6 Change 'git' to 'Git' whenever the whole system is referred to #3
  59b26cb Change 'git' to 'Git' whenever the whole system is referred to #2
  e458530 Change 'git' to 'Git' whenever the whole system is referred to #1
  c8fc93a Documentation: update two leftover small caps
  170a796 Documentation: avoid poor-man's small caps

I will locally squash the top one (your #6) into the bottom one
(your #1).  Perhaps people can proofread the result of Some
instances of 'git' should be 'Git' as they talk about the system
patches, and send in incremental corrections to be applied on top.

--
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


Aw: Re: Aw: Re: [PATCH v3 1/6] Change old system name 'GIT' to 'Git'

2013-01-22 Thread Thomas Ackermann
 
 
 I left this as-is in my fixup commit, as the html and manpage
 backends seem to typeset the section titles differently.  I however
 have to wonder what the result will look like if we replaced it with
 
   Typical use of Git credential
 
 Can you try and see how well it works?
 
The html and manpage backends both don't change the 'poor-man's bold face'
of the section titles and all the rest of the documentation uses this same 
style.
So we should leave this as is.

 
 I'd really hate to see these patches posted again to the list and
 asking for re-reviewing the whole thing.  Can we go incremental
 while parking these original patches on 'pu', and later squash the
 whole thing into two commits, or something?
 
Of course.


---
Thomas
--
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 v3 1/6] Change old system name 'GIT' to 'Git'

2013-01-21 Thread Junio C Hamano
Thomas Ackermann th.ac...@arcor.de writes:

 Git changed its 'official' system name from 'GIT' to 'Git' in v1.6.5.3
 (as can be seen in the corresponding release note where 'GIT' was
 changed to 'Git' in the header line). So change every occurrence
 of 'GIT in the documention to 'Git'.

 Signed-off-by: Thomas Ackermann th.ac...@arcor.de
 ---

This is more about stop using poor-man's small caps.

I think it misses GIT - the stupid content tracker in README, but
probably it is OK (it is not an end-user facing documentation).

I noticed that these two places still use poor-man's small caps
after this patch.

 * Documentation/SubmittingPatches:
 that are being emailed around. Although core GIT is a lot

 * Documentation/git-credential.txt:
 TYPICAL USE OF GIT CREDENTIAL

With these two updated, I think it is a sensible change and the
patch is timed reasonably well.  It applies to 'maint', and other
topics in flight do not seem to add new uses of GIT.

Not commenting on other 5 patches in the series yet, but if they
interact with other topics in flight, they may have to be separated
out. We'll see.

Thanks.
--
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