Re: [PATCH] submodule: make 'show' an alias for 'summary'

2012-10-01 Thread Jens Lehmann
Am 01.10.2012 19:33, schrieb Ramkumar Ramachandra: > Jens Lehmann wrote: >> Am 01.10.2012 08:45, schrieb Ramkumar Ramachandra: >>> Jens Lehmann wrote: I'm very interested in your feedback as a first time submodule user, what you wrote above makes sense and explains why you did that patch

Re: [PATCH] submodule: make 'show' an alias for 'summary'

2012-10-01 Thread Junio C Hamano
Jens Lehmann writes: > Me too would expect a show command to show me a list of all the > submodules and maybe some extra information (is it populated or > not, does it have its .git directory embedded, does it contain > changes). So maybe "show" should be a slightly pimped "status"? What's the n

Re: [PATCH] submodule: make 'show' an alias for 'summary'

2012-10-01 Thread Ramkumar Ramachandra
Hi Jens, Jens Lehmann wrote: > Am 01.10.2012 08:45, schrieb Ramkumar Ramachandra: >> Jens Lehmann wrote: >>> I'm very interested in your feedback as a first time submodule user, >>> what you wrote above makes sense and explains why you did that patch >>> (and it would have been nice to read some o

Re: [PATCH] submodule: make 'show' an alias for 'summary'

2012-10-01 Thread Jens Lehmann
Am 01.10.2012 08:45, schrieb Ramkumar Ramachandra: > Jens Lehmann wrote: >> I'm very interested in your feedback as a first time submodule user, >> what you wrote above makes sense and explains why you did that patch >> (and it would have been nice to read some of it in the commit message >> ;-). W

Re: [PATCH] submodule: make 'show' an alias for 'summary'

2012-09-30 Thread Ramkumar Ramachandra
Hi Jens, Jens Lehmann wrote: > I'm very interested in your feedback as a first time submodule user, > what you wrote above makes sense and explains why you did that patch > (and it would have been nice to read some of it in the commit message > ;-). What information did you expect to get from a "g

Re: [PATCH] submodule: make 'show' an alias for 'summary'

2012-09-29 Thread Jens Lehmann
Am 29.09.2012 17:07, schrieb Ramkumar Ramachandra: > Jens Lehmann wrote: >> Am 29.09.2012 15:43, schrieb Ramkumar Ramachandra: >>> Signed-off-by: Ramkumar Ramachandra >>> --- >>> Like 'git remote show', 'git stash show'. >> >> I understand the analogy to "git stash show" (as that also displays >>

Re: [PATCH] submodule: make 'show' an alias for 'summary'

2012-09-29 Thread Ramkumar Ramachandra
Hi Jens, Jens Lehmann wrote: > Am 29.09.2012 15:43, schrieb Ramkumar Ramachandra: >> Signed-off-by: Ramkumar Ramachandra >> --- >> Like 'git remote show', 'git stash show'. > > I understand the analogy to "git stash show" (as that also displays > a diff similar to what " git submodule summary" d

Re: [PATCH] submodule: make 'show' an alias for 'summary'

2012-09-29 Thread Jens Lehmann
Am 29.09.2012 15:43, schrieb Ramkumar Ramachandra: > Signed-off-by: Ramkumar Ramachandra > --- > Like 'git remote show', 'git stash show'. I understand the analogy to "git stash show" (as that also displays a diff similar to what " git submodule summary" does). But "git remote show" just display

[PATCH] submodule: make 'show' an alias for 'summary'

2012-09-29 Thread Ramkumar Ramachandra
Signed-off-by: Ramkumar Ramachandra --- Like 'git remote show', 'git stash show'. Documentation/git-submodule.txt |3 ++- git-submodule.sh|9 - 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/Documentation/git-submodule.txt b/Documentation/git-sub