[PATCH 0/3] archive: abbreviate substituted commit ids again

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Jul 27, 2010, 18:29 UTC
Message-ID: <20100727182942.GB5578@burratino>
In-Reply-To: <7vzkxc7rpn.fsf@alter.siamese.dyndns.org>

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