On Wed, Oct 10, 2007 at 01:16:03PM +0200, Jan Wielemaker wrote: > + # if we have a -p we should just send the file > + if ( exists ( $state->{opt}{p} ) ) > + { > + if ( open my $fh, '-|', "git-cat-file", "blob", $meta->{filehash} ) > + { while ( <$fh> ) > + { print "M " . $_; > + } > + close $fh or die ("Couldn't close filehandle for transmitfile(): $!"); > + } else > + { die("Couldn't execute git-cat-file"); > + } > + > + next; > + } There seems to be inconsistent whitespace in the patch. And please never do that else\n{ again, it hurts my eye ;) Will try to test (and write a testcase for) it tomorrow. Gruesse, -- Frank Lichtenheld <frank@xxxxxxxxxxxxxx> www: http://www.djpig.de/ - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html