git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] git-svn: un-break "git svn rebase" when log.abbrevCommit=true

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Jan 29, 2012, 02:59 UTC
Message-ID
<20120129025914.GA5981@burratino>
In-Reply-To
<1327803073-7000-1-git-send-email-avarab@gmail.com>
Ævar Arnfjörð Bjarmason wrote:
Show 9 quoted lines
> --- a/git-svn.perl
> +++ b/git-svn.perl
> @@ -1879,7 +1879,8 @@ sub cmt_sha2rev_batch {
>  sub working_head_info {
>  	my ($head, $refs) = @_;
>  	my @args = qw/log --no-color --no-decorate --first-parent
> -	              --pretty=medium/;
> +	              --pretty=medium --no-abbrev-commit --no-decorate/;
> +
Oh, dear.  Wouldn't
	my @args = qw/rev-list --first-parent --pretty=medium/;
be simpler and more robust as a quick fix?

Of course something that takes plain rev-list --first-parent output and pipes it through to cat-file --batch might be more intuitive, but rev-list --pretty should at least work. ;-)

Previous: Ævar Arnfjörð BjarmasonNext: Ævar Arnfjörð Bjarmason
Message 2 of 9 in “git-svn: un-break "git svn rebase" when log.abbrevCommit=true”
  1. git-svn: un-break "git svn rebase" when log.abbrevCommit=trueÆvar Arnfjörð Bjarmason, Jan 29, 2012
  2. Jonathan NiederJan 29, 2012
  3. Ævar Arnfjörð BjarmasonJan 29, 2012
  4. Dan JohnsonJan 30, 2012
  5. Ævar Arnfjörð BjarmasonJan 30, 2012
  6. 1/2 git-svn: remove redundant porcelain option to rev-listÆvar Arnfjörð Bjarmason, Feb 12, 2012
  7. 2/2 git-svn: un-break "git svn rebase" when log.abbrevCommit=trueÆvar Arnfjörð Bjarmason, Feb 12, 2012
  8. Jonathan NiederFeb 12, 2012
  9. Eric WongFeb 12, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.