Re: [PATCH v3 05/10] submodule: decouple url and submodule existence

2017-03-15 Thread Brandon Williams
On 03/14, Junio C Hamano wrote: > Brandon Williams writes: > > > Currently the submodule..url config option is used to determine > > if a given submodule exists and is interesting to the user. This > > however doesn't work very well because the URL is a config option for > > the scope of a repos

Re: [PATCH v3 05/10] submodule: decouple url and submodule existence

2017-03-15 Thread Brandon Williams
On 03/13, Stefan Beller wrote: > + cc Jens, FYI. > > Once upon a time I brought up different addressing/activating mechanism for > submodules and I remember Jens having some uneasy thoughts about > that back in the day. This series addresses the user confusion and > documentation > better than wh

Re: [PATCH v3 05/10] submodule: decouple url and submodule existence

2017-03-14 Thread Junio C Hamano
Junio C Hamano writes: > Also submodule.active that takes pathspec and not name is an oddball > (use of "name" not "path" is to prepare for a submodule whose > location in the superproject changes depending on the commit in the > superproject), and we need to justify with an explanation. I think

Re: [PATCH v3 05/10] submodule: decouple url and submodule existence

2017-03-14 Thread Junio C Hamano
Brandon Williams writes: > Currently the submodule..url config option is used to determine > if a given submodule exists and is interesting to the user. This > however doesn't work very well because the URL is a config option for > the scope of a repository, whereas the existence of a submodule

Re: [PATCH v3 05/10] submodule: decouple url and submodule existence

2017-03-13 Thread Stefan Beller
+ cc Jens, FYI. Once upon a time I brought up different addressing/activating mechanism for submodules and I remember Jens having some uneasy thoughts about that back in the day. This series addresses the user confusion and documentation better than what I had back then. On Mon, Mar 13, 2017 at 2

[PATCH v3 05/10] submodule: decouple url and submodule existence

2017-03-13 Thread Brandon Williams
Currently the submodule..url config option is used to determine if a given submodule exists and is interesting to the user. This however doesn't work very well because the URL is a config option for the scope of a repository, whereas the existence of a submodule is an option scoped to the working