Re: [PATCH] clone: fix repo name when cloning a server's root

2015-07-27 Thread Junio C Hamano
Duy Nguyen writes: >> I was not able to come by with a useful test as that would >> require being able to clone a root directory. I couldn't find >> anything in the current tests that looks like what I want to do. >> Does anybody have an idea on how to achieve this? > > There's t/t1509/prepare-ch

Re: [PATCH] clone: fix repo name when cloning a server's root

2015-07-27 Thread Patrick Steinhardt
On Mon, Jul 27, 2015 at 07:51:30PM +0700, Duy Nguyen wrote: > On Mon, Jul 27, 2015 at 6:48 PM, Patrick Steinhardt wrote: > > When cloning a repository from a server's root, that is the URL's > > path component is a '/' only, we fail to generate a sensible > > repository name when the URL contains

Re: [PATCH] clone: fix repo name when cloning a server's root

2015-07-27 Thread Duy Nguyen
On Mon, Jul 27, 2015 at 6:48 PM, Patrick Steinhardt wrote: > When cloning a repository from a server's root, that is the URL's > path component is a '/' only, we fail to generate a sensible > repository name when the URL contains authentication data. This > is especially bad when cloning URLs like

[PATCH] clone: fix repo name when cloning a server's root

2015-07-27 Thread Patrick Steinhardt
When cloning a repository from a server's root, that is the URL's path component is a '/' only, we fail to generate a sensible repository name when the URL contains authentication data. This is especially bad when cloning URLs like 'ssh://user:pas...@example.com/', which results in a repository 'pa