Re: submodule-summary

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

Re: submodule-summary

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:47:33

Jens Lehmann [off-list ref] writes:
Good point. (Personally i like the options --submodule=shortlog and
--submodule=twoline. Because IMHO --submodule=summary could make
people expect similar output to git submodule summary, no?).
Well, _that_ is something we could fix in "git submodule", isn't it?

Right now, the only way "git submodule" command can summarize the changes
is by showing a left-right shortlog, but it wouldn't be so inconceivable
that somebody may come up with different and equally useful way to show
them, e.g. diffstat, and at that point, we would probably want to have
"git submodule summary --stat" or "git submodule summary --shortlog" (and
the default being --shortlog).

I am _not_ married to the naming "shortlog", by the way, and shortlog is
rather a bad name for it.  Sorry for suggesting it; it is quite different
from the actual "git shortlog" command output (and no I am not suggesting
to make the output similar to shortlog), but rather is more similar to
"log --left-right --oneline".  But I think you got the point.

Re: submodule-summary

From: Jens Lehmann <hidden>
Date: 2016-06-15 22:47:33

Junio C Hamano schrieb:
I am _not_ married to the naming "shortlog", by the way, and shortlog is
rather a bad name for it.  Sorry for suggesting it; it is quite different
from the actual "git shortlog" command output (and no I am not suggesting
to make the output similar to shortlog), but rather is more similar to
"log --left-right --oneline".  But I think you got the point.
I have no strong feelings about the option names either. What about
"--submodule=short" for the current default git diff output and
"--submodule=left-right-log" for the one resembling the current output
of submodule summary? I really don't know, if there are better proposals
i'll happily use them.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help