From: Junio C Hamano Date: Sat, 27 Nov 2010 06:33:15 GMT Subject: Re: [PATCH 1/3] cvsimport: partial whitespace cleanup Message-ID: <7v8w0fp978.fsf@alter.siamese.dyndns.org> In-Reply-To: <939dff25f5a58b6f37cf21bf490b2782f6e96f39.1290697830.git.git@drmicha.warpmail.net> Michael J Gruber writes: > @@ -100,13 +100,13 @@ sub read_repo_config { > my $arg = 'git config'; > $arg .= ' --bool' if ($o !~ /:$/); > > - chomp(my $tmp = `$arg --get cvsimport.$key`); > + chomp(my $tmp = `$arg --get cvsimport.$ckey`); At this point in the series you haven't introduced $ckey yet, hence this will break, no?