Re: [PATCHv2 04/10] remote: Reject remote names containing '/'

2013-05-16 Thread Ramkumar Ramachandra
Johan Herland wrote: > FWIW, I've abandoned this patch, and don't care much about multi-level > remote names anymore. They work in current git, and they will sort-of > work with remote ref namespaces as well, although you will have to use > full refnames when referring to their remote-tracking refs

Re: [PATCHv2 04/10] remote: Reject remote names containing '/'

2013-05-16 Thread Johan Herland
On Thu, May 16, 2013 at 11:48 AM, Ramkumar Ramachandra wrote: > Johan Herland wrote: >> The disambiguation can probably be resolved, although the resulting >> code will obviously be somewhat more cumbersome and ugly (and IMHO the >> current code is plenty of that already...). Combine this with the

Re: [PATCHv2 04/10] remote: Reject remote names containing '/'

2013-05-16 Thread Ramkumar Ramachandra
Johan Herland wrote: > The disambiguation can probably be resolved, although the resulting > code will obviously be somewhat more cumbersome and ugly (and IMHO the > current code is plenty of that already...). Combine this with the > problems of clobbering of the same remote-tracking ref (describe

Re: [PATCHv2 04/10] remote: Reject remote names containing '/'

2013-05-12 Thread Johan Herland
On Mon, May 13, 2013 at 6:54 AM, Junio C Hamano wrote: > Johan Herland writes: > >> Although we definitely support and encourage use of multi-level branch >> names, we have never conciously tried to give support for multi-level >> remote names. Currently, they are allowed, but there is no evidenc

Re: [PATCHv2 04/10] remote: Reject remote names containing '/'

2013-05-12 Thread Johan Herland
On Mon, May 13, 2013 at 6:48 AM, Eric Sunshine wrote: > On Sat, May 11, 2013 at 12:21 PM, Johan Herland wrote: >> Although we definitely support and encourage use of multi-level branch >> names, we have never conciously tried to give support for multi-level > > s/conciously/consciously/ Thanks,

Re: [PATCHv2 04/10] remote: Reject remote names containing '/'

2013-05-12 Thread Junio C Hamano
Johan Herland writes: > Although we definitely support and encourage use of multi-level branch > names, we have never conciously tried to give support for multi-level > remote names. Currently, they are allowed, but there is no evidence that > they are commonly used. > > Now, they do provide a so

Re: [PATCHv2 04/10] remote: Reject remote names containing '/'

2013-05-12 Thread Eric Sunshine
On Sat, May 11, 2013 at 12:21 PM, Johan Herland wrote: > Although we definitely support and encourage use of multi-level branch > names, we have never conciously tried to give support for multi-level s/conciously/consciously/ > remote names. Currently, they are allowed, but there is no evidence

[PATCHv2 04/10] remote: Reject remote names containing '/'

2013-05-11 Thread Johan Herland
Although we definitely support and encourage use of multi-level branch names, we have never conciously tried to give support for multi-level remote names. Currently, they are allowed, but there is no evidence that they are commonly used. Now, they do provide a source of problems when trying to exp