Re: [PATCH 3/3] git-svn: Reduce temp file usage when dealing with non-links
- From
- Eric Wong <normalperson@yhbt.net>
- Date
- Aug 13, 2008, 03:52 UTC
- Message-ID
- <20080813035211.GA31792@untitled>
- In-Reply-To
- <48A2583A.1000509@griep.us>
Marcus Griep <marcus@griep.us> wrote:
Show 20 quoted lines
> Eric Wong wrote:
> > Thank you Marcus!
> >
> > Acked-by: Eric Wong <normalperson@yhbt.net>
>
> Errr, you want to Ack [PATCH v2 3/3]; not this one; there's one minor
> typo when I shortened the variable $result to $res.
>
> >> + my $res;
> >> + while ($res = sysread($fh, my $str, 1024)) {
> >> + my $out = syswrite($tmp_fh, $str, $res);
> >> + defined($out) && $out == $res
> >> + or croak("write ",
> >> + $tmp_fh->filename,
> >> + ": $!\n");
> >> + }
> >> + defined $result or croak $!;
>
> That last line causes compilation errors with 'use strict'. It is fixed
> in the alternate version.Oops, I applied the right patch but managed to reopen the wrong one when I acked it.
I've just pushed out my repository with acks to git://bogomips.org/git-svn.git
-- Eric Wong