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

Re: [PATCH] git-svnimport.perl: fix for 'arg list too long...'

From
SKSasha Khapyorsky <sashak@voltaire.com>
Date
Feb 2, 2006, 14:26 UTC
Message-ID
<20060202142638.GD32188@sashak.voltaire.com>
In-Reply-To
<7v3bj27kxe.fsf@assigned-by-dhcp.cox.net>
On 14:50 Wed 01 Feb     , Junio C Hamano wrote:
Show 5 quoted lines
> 
> *1* I do not think this makes much of a difference but here is what
> I mean.
> 
> diff --git a/git-svnimport.perl b/git-svnimport.perl
Tested. Works good. I think your version is cleaner. Cosmetic comment btw:
Show 5 quoted lines
> +		open my $F, "|-",
> +			qw(git-update-index --force-remove -z --stdin)
> +				or die $!;
> +		print $F "$_\0" for @u;
> +		close $F or die $!;

It is not critical, but this block may be under if(@u) {..} to prevent "empty" git-update-index invocations.

Sasha.
Previous: Matthias Urlichs
Message 6 of 6 in “git-svnimport.perl: fix for 'arg list too long...'”
  1. git-svnimport.perl: fix for 'arg list too long...'Sasha Khapyorsky, Feb 1, 2006
  2. Junio C HamanoFeb 1, 2006
  3. Sasha KhapyorskyFeb 1, 2006
  4. Junio C HamanoFeb 1, 2006
  5. Matthias UrlichsFeb 1, 2006
  6. Sasha KhapyorskyFeb 2, 2006

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.