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

Re: [spf:guess] Re: [PATCH 1/2] git-svn: ignore changeless commits when checking for a cherry-pick

From
Sam Vilain <sam@vilain.net>
Date
Jan 6, 2010, 22:38 UTC
Message-ID
<4B4510EE.4090504@vilain.net>
In-Reply-To
<20100106214338.GA5115@dcvr.yhbt.net>
Eric Wong wrote:
Show 18 quoted lines
> Andrew Myrick <amyrick@apple.com> wrote:
>   
>> diff --git a/git-svn.perl b/git-svn.perl
>> index 650c9e5..8c7c034 100755
>> --- a/git-svn.perl
>> +++ b/git-svn.perl
>> @@ -3052,12 +3052,36 @@ sub check_cherry_pick {
>>  	for my $range ( @ranges ) {
>>  		delete @commits{_rev_list($range)};
>>  	}
>> +        for my $commit (keys %commits) {
>>     
>
> Hi Andrew,
>
> I'll again defer to Sam for Acks on these.  Test cases would be nice to
> have, too.
>   

They look fine to me, agreed a test case would be nice and make sure the features aren't lost later inadvertently.

Thanks Andrew, Sam

Previous: Eric WongNext: Eric Wong
Message 4 of 11 in “git-svn: ignore changeless commits when checking for a cherry-pick”
  1. 1/2 git-svn: ignore changeless commits when checking for a cherry-pickAndrew Myrick, Jan 6, 2010
  2. 2/2 git-svn: handle merge-base failuresAndrew Myrick, Jan 6, 2010
  3. Eric WongJan 6, 2010
  4. Sam VilainJan 6, 2010
  5. Eric WongJan 7, 2010
  6. Sam VilainJan 7, 2010
  7. Andrew MyrickJan 6, 2010
  8. 1/2 git-svn: ignore changeless commits when checking for a cherry-pickAndrew Myrick, Jan 7, 2010
  9. 2/2 git-svn: handle merge-base failuresAndrew Myrick, Jan 7, 2010
  10. Eric WongJan 7, 2010
  11. git-svn: update svn mergeinfo test suiteAndrew Myrick, Jan 21, 2010

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.