Re: [PATCH] Disown ssh+git and git+ssh

2016-03-24 Thread Carlos Martín Nieto
On Wed, 2016-03-09 at 13:56 -0800, Junio C Hamano wrote: > Eric Sunshine writes: > > > > > It might be helpful to cite some reference to support the claim > > that > > they are "silly" since it's not necessarily obvious to readers who > > did > > not following the discussion. > > ... > > > > >

Re: [PATCH] Disown ssh+git and git+ssh

2016-03-10 Thread Eric Sunshine
On Wed, Mar 9, 2016 at 4:56 PM, Junio C Hamano wrote: > Eric Sunshine writes: >> It might be helpful to cite some reference to support the claim that >> they are "silly" since it's not necessarily obvious to readers who did >> not following the discussion. >> ... >>> || starts_wit

Re: [PATCH] Disown ssh+git and git+ssh

2016-03-09 Thread Junio C Hamano
Eric Sunshine writes: > It might be helpful to cite some reference to support the claim that > they are "silly" since it's not necessarily obvious to readers who did > not following the discussion. > ... >> || starts_with(url, "ssh://") >> + /* >> +*

Re: [PATCH] Disown ssh+git and git+ssh

2016-02-15 Thread Eric Sunshine
On Mon, Feb 15, 2016 at 9:29 AM, Carlos Martín Nieto wrote: > These were silly from the beginning, but we have to support them for It might be helpful to cite some reference to support the claim that they are "silly" since it's not necessarily obvious to readers who did not following the discussi

Re: [PATCH] Disown ssh+git and git+ssh

2016-02-13 Thread Junio C Hamano
Carlos Martín Nieto writes: > These were silly from the beginning, but we have to support them for > compatibility. That doesn't mean we have to show them in the > documentation. These were already left out of the main list, but a > reference in the main manpage was left, so remove that. > > Also

Re: [PATCH] Disown ssh+git and git+ssh

2016-02-12 Thread Jeff King
On Fri, Feb 12, 2016 at 04:09:37PM +0100, Carlos Martín Nieto wrote: > These were silly from the beginning, but we have to support them for > compatibility. That doesn't mean we have to show them in the > documentation. These were already left out of the main list, but a > reference in the main ma