Re: [PATCH 0/3] recursive support for ls-files

2016-09-26 Thread Junio C Hamano
Brandon Williams writes: > or something along those lines. That way we only need to add the flag > to each command as we go ... Sounds good. Thanks.

Re: [PATCH 0/3] recursive support for ls-files

2016-09-26 Thread Brandon Williams
On 09/26, Junio C Hamano wrote: > Brandon Williams writes: > > > In a previous email you mentioned that this feature should be completely > > hidden from users, which is why I removed the command line option for > > this latest series. > > I may have said something like that;

Re: [PATCH 0/3] recursive support for ls-files

2016-09-26 Thread Junio C Hamano
Brandon Williams writes: > In a previous email you mentioned that this feature should be completely > hidden from users, which is why I removed the command line option for > this latest series. I may have said something like that; I do not recall, though, so a more accurate

Re: [PATCH 0/3] recursive support for ls-files

2016-09-26 Thread Brandon Williams
On 09/25, Junio C Hamano wrote: > Brandon Williams writes: > > > On 09/25, Jeff King wrote: > >> On Fri, Sep 23, 2016 at 05:13:31PM -0700, Brandon Williams wrote: > >> > >> > After looking at the feedback I rerolled a few things, in particular the > >> > --submodule_prefix

Re: [PATCH 0/3] recursive support for ls-files

2016-09-25 Thread Junio C Hamano
Brandon Williams writes: > On 09/25, Jeff King wrote: >> On Fri, Sep 23, 2016 at 05:13:31PM -0700, Brandon Williams wrote: >> >> > After looking at the feedback I rerolled a few things, in particular the >> > --submodule_prefix option that existed to give a submodule context

Re: [PATCH 0/3] recursive support for ls-files

2016-09-25 Thread Brandon Williams
On 09/25, Jeff King wrote: > On Fri, Sep 23, 2016 at 05:13:31PM -0700, Brandon Williams wrote: > > > After looking at the feedback I rerolled a few things, in particular the > > --submodule_prefix option that existed to give a submodule context about > > where > > it had been invoked from.

Re: [PATCH 0/3] recursive support for ls-files

2016-09-25 Thread Jeff King
On Fri, Sep 23, 2016 at 05:13:31PM -0700, Brandon Williams wrote: > After looking at the feedback I rerolled a few things, in particular the > --submodule_prefix option that existed to give a submodule context about where > it had been invoked from. People didn't seem to like the idea of

[PATCH 0/3] recursive support for ls-files

2016-09-23 Thread Brandon Williams
After looking at the feedback I rerolled a few things, in particular the --submodule_prefix option that existed to give a submodule context about where it had been invoked from. People didn't seem to like the idea of exposing this to the users (yet anyways) so I removed it as an option and