[PATCH v2 0/5] jn/shortlog
- From
Jonathan Nieder <jrnieder@gmail.com>
- Date
- May 4, 2010, 02:52 UTC
- Message-ID
- <20100504025217.GA7322@progeny.tock>
- In-Reply-To
- <20100504015635.GA7258@progeny.tock>
Jonathan Nieder wrote:
Show 7 quoted lines
> apparently shortlog does not default to HEAD when stdin is not a > tty. > > Possible fixes: > > - document the current unfriendly behavior > - default to HEAD when input is empty
Here’s the uncontroversial half.
Patch 1 is new: it explains why scripts that mean HEAD need to explicitly say so. The remaining patches are just like before, except tests pass a HEAD arg now. Sorry about the breakage, and thanks for catching it.
Jonathan Nieder (4):
Documentation/shortlog: scripted users should not rely on implicit
HEAD
t4201 (shortlog): guard setup with test_expect_success
t4201 (shortlog): Test output format with multiple authors
shortlog: Document and test --format optionWill Palmer (1): pretty: Respect --abbrev option
Documentation/git-shortlog.txt | 15 +++++- builtin/shortlog.c | 3 +- pretty.c | 7 ++- shortlog.h | 1 + t/t4201-shortlog.sh | 116 +++++++++++++++++++++++++++++++-------- t/t6006-rev-list-format.sh | 19 +++++++ 6 files changed, 132 insertions(+), 29 deletions(-)