Re: [REQUEST 1/1] docs: update http..* options documentation

2013-07-26 Thread Kyle J. McKay
On Jul 26, 2013, at 19:43, Jeff King wrote: On Fri, Jul 26, 2013 at 07:15:07PM -0700, Kyle J. McKay wrote: So there's a version of this in next as cea9928 and I think that version is fine. If the documentation gets updated in the future, or perhaps moved to a separate urls-matching.txt for som

Re: [REQUEST 1/1] docs: update http..* options documentation

2013-07-26 Thread Jeff King
On Fri, Jul 26, 2013 at 07:15:07PM -0700, Kyle J. McKay wrote: > So there's a version of this in next as cea9928 and I think that > version is fine. If the documentation gets updated in the future, or > perhaps moved to a separate urls-matching.txt for some reason at some > point (git config --ur

Re: [REQUEST 1/1] docs: update http..* options documentation

2013-07-26 Thread Kyle J. McKay
On Jul 26, 2013, at 15:27, Jeff King wrote: On Thu, Jul 25, 2013 at 11:23:23PM -0700, Kyle J. McKay wrote: IMHO, this would be more clear as a single item, like: . User name (e.g., `user` in `https://u...@example.com/repo.git`). If the config key has a user name it must match the user nam

Re: [REQUEST 1/1] docs: update http..* options documentation

2013-07-26 Thread Jeff King
On Thu, Jul 25, 2013 at 11:23:23PM -0700, Kyle J. McKay wrote: > >IMHO, this would be more clear as a single item, like: > > > > . User name (e.g., `user` in `https://u...@example.com/repo.git`). If > > the config key has a user name it must match the user name in the > > URL exactly. If the c

Re: [REQUEST 1/1] docs: update http..* options documentation

2013-07-25 Thread Kyle J. McKay
On Jul 25, 2013, at 23:23, Kyle J. McKay wrote: On Jul 25, 2013, at 21:37, Jeff King wrote: You should add your S-O-B, too, for your bits. Yes. I wanted to make certain it didn't get applied just yet. :) Looks like it got patch snarfed anyway. ;-) On Jul 25, 2013, at 23:07, Junio C Hamano

Re: [REQUEST 1/1] docs: update http..* options documentation

2013-07-25 Thread Kyle J. McKay
On Jul 25, 2013, at 21:37, Jeff King wrote: On Thu, Jul 25, 2013 at 03:39:13PM -0700, Kyle J. McKay wrote: Overhaul the text of the http..* options documentation providing a hopefully easier to understand itemized list of matching behavior as suggested by and including text from Jeff King. ---

Re: [REQUEST 1/1] docs: update http..* options documentation

2013-07-25 Thread Junio C Hamano
Junio C Hamano writes: > Will queue the attached, unless you spot glaring errors made by me, > coding while drunk ;-) ... aand, it turns out that I screwed up while running "git commit --amend" (missed "-a"). I should go to bed now. -- >8 -- From: "Kyle J. McKay" Date: Thu, 25 Jul 2013 15

Re: [REQUEST 1/1] docs: update http..* options documentation

2013-07-25 Thread Junio C Hamano
Jeff King writes: > Signed-off-by: Jeff King > > You should add your S-O-B, too, for your bits. > ... > So this patch looks pretty good to me; the two points I raised above are > how I would have done it, but they are relatively minor if you do not > agree. Will queue the attached, unless you s

Re: [REQUEST 1/1] docs: update http..* options documentation

2013-07-25 Thread Jeff King
On Thu, Jul 25, 2013 at 03:39:13PM -0700, Kyle J. McKay wrote: > Overhaul the text of the http..* options documentation > providing a hopefully easier to understand itemized list of > matching behavior as suggested by and including text from > Jeff King. > --- Signed-off-by: Jeff King You shoul

[REQUEST 1/1] docs: update http..* options documentation

2013-07-25 Thread Kyle J. McKay
Overhaul the text of the http..* options documentation providing a hopefully easier to understand itemized list of matching behavior as suggested by and including text from Jeff King. --- Documentation/config.txt | 60 +--- 1 file changed, 37 insertions(