Junio C Hamano wrote: > isn't > this one in builtin/commit.c also a confusing no-op? I see. When setup_revisions is called (which initializes diffopt), rev.abbrev still equals DEFAULT_ABBREV. I missed v1.7.1.1~17^2 (commit::print_summary(): don't use format_commit_message(), 2010-06-12) and did not notice that the bug had gone away. Sorry for the confusion. Here’s a rerolled series. Jonathan Nieder (3): archive: abbreviate substituted commit ids again checkout, commit: remove confusing assignments to rev.abbrev examples/commit: use --abbrev for commit summary archive.c | 1 + builtin/checkout.c | 1 - builtin/commit.c | 1 - contrib/examples/git-commit.sh | 2 +- t/t5001-archive-attr.sh | 2 +- 5 files changed, 3 insertions(+), 4 deletions(-) -- 1.7.2.21.g04ff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html