threads / patch / 31301

patchgit-p4: Strip branch name prefixes if --use-client-spec enabled

Subject: [PATCH] git-p4: Strip branch name prefixes if --use-client-spec enabled

## tl;dr

2 messages between Aug 20, 2012 and Aug 22, 2012. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Patrick C. McGinty· Aug 20, 2012, 23:43 UTC · lore
From: "Patrick C. McGinty" <casey.mcginty@gmail.com>

This removes the branch names from being included in the file system path when using the both --use-client-spec and --detect-branches options.

Used alone, the --detect-branches option did the right thing, removing the branch names from the relative git path. By adding the --use-client-spec option, the old logic would incorrectly bypass this step.

Signed-off-by: Patrick C. McGinty <casey.mcginty@gmail.com>
---
 git-p4.py |   10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)
Show changes to git-p4.py +9 −1
diff --git a/git-p4.py b/git-p4.py
index e67d37d..9489845 100755
--- a/git-p4.py
+++ b/git-p4.py
@@ -1819,7 +1819,15 @@ class P4Sync(Command, P4UserMap):
 
     def stripRepoPath(self, path, prefixes):
         if self.useClientSpec:
-            return self.clientSpecDirs.map_in_client(path)
+            # map path to user defined client location
+            path = self.clientSpecDirs.map_in_client(path)
+            # Remove prefixes (e.g. branch names) from path. Apply the
+            # clientSpecDirs map to the prefix paths to match the
+            # modified path value.
+            for p in map(self.clientSpecDirs.map_in_client, prefixes):
+               if p and p4PathStartsWith( path, p):
+                  path = path[len(p):]
+            return path
 
         if self.keepRepoPath:
             prefixes = [re.sub("^(//[^/]+/).*", r'\1', prefixes[0])]
-- 
1.7.9.5
Pete Wyckoff· Aug 22, 2012, 21:22 UTC · re: Patrick C. McGinty · lore

Re: [PATCH] git-p4: Strip branch name prefixes if --use-client-spec enabled

casey.mcginty@gmail.com wrote on Mon, 20 Aug 2012 13:43 -1000:
Show 8 quoted lines
> This removes the branch names from being included in the file system
> path when using the both --use-client-spec and --detect-branches
> options.
> 
> Used alone, the --detect-branches option did the right thing, removing
> the branch names from the relative git path. By adding the
> --use-client-spec option, the old logic would incorrectly bypass this
> step.

Thanks for reporting this bug, and supplying a fix. I think, though that this is the same issue we were looking at in this thread:

    http://article.gmane.org/gmane.comp.version-control.git/203294

It is queued up and the fix likely will appear in 1.7.13 (or whatever the next release after 1.7.12 is called).

(Gmane has been down all day from here; there's information from another reporter in http://stackoverflow.com/questions/11893688/ .)

		-- Pete

← back to recent threads