Re: [RFC/WIP PATCH 0/4] less ignorance of submodules for ignore=all

2013-12-09 Thread Junio C Hamano
Jens Lehmann writes: > I didn't check each site in detail, but I suspect each ignore option > was added on purpose to fix a problem. That means we still need "all" > (at least when diffing rev<->index). Unfortunately that area is not > covered well in our tests, I only got breakage from the filte

Re: Re: [RFC/WIP PATCH 0/4] less ignorance of submodules for ignore=all

2013-12-09 Thread Heiko Voigt
On Thu, Dec 05, 2013 at 09:51:31PM +0100, Jens Lehmann wrote: > Am 05.12.2013 00:19, schrieb Heiko Voigt: > > On Wed, Dec 04, 2013 at 02:32:46PM -0800, Junio C Hamano wrote: > > This series tries to achieve the following goals for the > > submodule..ignore=all configuration or the --ignore-submodul

Re: [RFC/WIP PATCH 0/4] less ignorance of submodules for ignore=all

2013-12-05 Thread Jens Lehmann
Am 05.12.2013 00:19, schrieb Heiko Voigt: > On Wed, Dec 04, 2013 at 02:32:46PM -0800, Junio C Hamano wrote: > This series tries to achieve the following goals for the > submodule..ignore=all configuration or the --ignore-submodules=all > command line switch. Thanks for the summary. > * Make git

Re: Re: [RFC/WIP PATCH 0/4] less ignorance of submodules for ignore=all

2013-12-04 Thread Heiko Voigt
On Wed, Dec 04, 2013 at 02:32:46PM -0800, Junio C Hamano wrote: > Heiko Voigt writes: > > > This is my current work in progress. Sergey it would be awesome if you > > could test these and tell me whether the behaviour is what you would > > expect. Once that is settled I will add some tests and po

Re: [RFC/WIP PATCH 0/4] less ignorance of submodules for ignore=all

2013-12-04 Thread Junio C Hamano
Heiko Voigt writes: > This is my current work in progress. Sergey it would be awesome if you > could test these and tell me whether the behaviour is what you would > expect. Once that is settled I will add some tests and possibly clean up > some code. > > Since nobody spoke against this change of

Re: [RFC/WIP PATCH 0/4] less ignorance of submodules for ignore=all

2013-12-04 Thread Heiko Voigt
On Wed, Dec 04, 2013 at 11:16:59PM +0100, Heiko Voigt wrote: > * The git diff $commit question Junio mentioned here[1] it does not yet >show diffs of ignore=all submodules. Forgot to add this here: [1] http://article.gmane.org/gmane.comp.version-control.git/238348 -- To unsubscribe from this

[RFC/WIP PATCH 0/4] less ignorance of submodules for ignore=all

2013-12-04 Thread Heiko Voigt
This is my current work in progress. Sergey it would be awesome if you could test these and tell me whether the behaviour is what you would expect. Once that is settled I will add some tests and possibly clean up some code. Since nobody spoke against this change of behavior I assume that we agree