Re: Cannot fetch whole history from Perforce

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Sorry for the slow response.

On Tue, Mar 8, 2011 at 2:33 PM, Tor Arvid Lund <torarvid@xxxxxxxxx> wrote:
> On Tue, Mar 8, 2011 at 9:36 PM, Thomas Berg <merlin66b@xxxxxxxxx> wrote:
>> Hi,
>>
>> On Tue, Mar 8, 2011 at 7:09 PM, Anatol Pomozov <anatol.pomozov@xxxxxxxxx> wrote:
>>> Hi, I am using HEAD version of the git and I have an issue with git-p4
>>> tool (p4 to git importer).
>>>
>>> git-p4 works fine when I import HEAD version of my project
>>>
>>> $ git p4 clone //depot/foo/bar --verbose
>>> But it fails with a cryptic error when I try to import @all revisions:
>> [...]
>>>
>>> $ git p4 clone //depot/foo/bar@all --verbose
>>> Importing from //depot/foo/bar@all into bar
>> [...]
>>>
>>> Can it be related to Pete's changes?
>>> http://permalink.gmane.org/gmane.comp.version-control.git/167281
>
> Whoops! The P4Clone class is a subclass of P4Sync, which means that
> this check (The one you mention from e32e00dc) will be done on every
> clone with @all. We can't have that :)
>
> It works for me if I patch it with this one-liner (but it's not
> exactly a beautiful looking line of code anymore):
>
> Â Â-- Tor Arvid
> ---
>
> 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),

Yep it fixes the problem for me. Do you have a plan to propose it for upstream?
--
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


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]