From: Pete Wyckoff Date: Fri, 11 Feb 2011 13:44:43 GMT Subject: Re: [PATCH] git-p4: Fix error message crash in P4Sync.commit. Message-ID: <20110211134443.GA11773@honk.padd.com> In-Reply-To: <1297167610-1795-1-git-send-email-torarvid@gmail.com> torarvid@gmail.com wrote on Tue, 08 Feb 2011 13:20 +0100: > There is an error message that crashes the script because of an invalid ref > to the non-existing "path" variable. It is almost never printed, which > would explain why nobody encountered this problem before... But anyway, > this oneliner fixes it. Needs signoff. Acked-by: Pete Wyckoff > --- > contrib/fast-import/git-p4 | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/contrib/fast-import/git-p4 b/contrib/fast-import/git-p4 > index ca3cea0..29b9e32 100755 > --- a/contrib/fast-import/git-p4 > +++ b/contrib/fast-import/git-p4 > @@ -1086,7 +1086,7 @@ class P4Sync(Command): > if [p for p in branchPrefixes if p4PathStartsWith(f['path'], p)]: > new_files.append (f) > else: > - sys.stderr.write("Ignoring file outside of prefix: %s\n" % path) > + sys.stderr.write("Ignoring file outside of prefix: %s\n" % f['path']) > > self.gitStream.write("commit %s\n" % branch) > # gitStream.write("mark :%s\n" % details["change"]) > -- > 1.7.3.1.68.g06779.dirty > > -- > To unsubscribe from this list: send the line "unsubscribe git" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >