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

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

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Feb 12, 2012, 00:31 UTC
Message-ID
<20120212003131.GG2719@burratino>
In-Reply-To
<1329006186-21346-2-git-send-email-avarab@gmail.com>
Ævar Arnfjörð Bjarmason wrote:
> Change git-svn to use git-rev-list(1) instead of git-log(1) since the
> latter is porcelain that'll cause "git svn rebase" to fail completely
> if log.abbrevCommit is set to true in the configuration.
[...]
Show 9 quoted lines
> --- a/git-svn.perl
> +++ b/git-svn.perl
> @@ -1878,8 +1878,7 @@ sub cmt_sha2rev_batch {
>  
>  sub working_head_info {
>  	my ($head, $refs) = @_;
> -	my @args = qw/log --no-color --no-decorate --first-parent
> -	              --pretty=medium/;
> +	my @args = qw/rev-list --first-parent --pretty=medium/;

Thanks! The other caller to "git log" in this script uses --pretty=raw and should be safe.

Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Previous: Ævar Arnfjörð BjarmasonNext: Eric Wong
Message 8 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.