From: Pete Wyckoff Date: Sat, 12 Mar 2011 16:23:23 GMT Subject: [PATCH 1/2] git-p4: fix clone @all regression Message-ID: <20110312162323.GB8509@arf.padd.com> In-Reply-To: <20110312162246.GA8509@arf.padd.com> Commit e32e00dc88948a730b8b1f3b8129f30c313713e7 "git-p4: better message for "git-p4 sync" when not cloned" broke the use of the "@all" revision specifier, e.g., "git-p4 clone //depot/xxx@all". Fix it as per Tor Arvid's quick patch. Signed-off-by: Pete Wyckoff Reported-by: Anatol Pomozov Based-on-patch-by: Tor Arvid Lund --- 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 2df3bb2..7cb479c 100755 --- a/contrib/fast-import/git-p4 +++ b/contrib/fast-import/git-p4 @@ -1763,7 +1763,7 @@ class P4Sync(Command): changes.sort() else: - if not self.p4BranchesInGit: + if not isinstance(self, P4Clone) and not self.p4BranchesInGit: die("No remote p4 branches. Perhaps you never did \"git p4 clone\" in here."); if self.verbose: print "Getting p4 changes for %s...%s" % (', '.join(self.depotPaths), -- 1.7.4.1