From: Lars Schneider larsxschnei...@gmail.com
Perforce depot may record paths in mixed cases, e.g. p4 files may
show that there are these two paths:
//depot/Path/to/file1
//depot/pATH/to/file2
and with p4 or p4v, these end up in the same directory, e.g.
//depot/Path/to/file1
//depot/Path/to/file2
which is the desired outcome on case insensitive systems.
If git-p4 is used with client spec //depot/Path/..., however, then
all files not matching the case in the client spec are ignored (in
the example above //depot/pATH/to/file2).
Fix this by using the path case that appears first in lexicographical
order when core.ignorcase is set to true. This behavior is consistent
with p4 and p4v.
Signed-off-by: Lars Schneider larsxschnei...@gmail.com
Acked-by: Luke Diamand l...@diamand.org
---
git-p4.py | 7 ++
t/t9821-git-p4-path-variations.sh | 200 ++
2 files changed, 207 insertions(+)
create mode 100755 t/t9821-git-p4-path-variations.sh
diff --git a/git-p4.py b/git-p4.py
index 073f87b..0093fa3 100755
--- a/git-p4.py
+++ b/git-p4.py
@@ -1950,10 +1950,14 @@ class View(object):
if unmap in res:
# it will list all of them, but only one not unmap-ped
continue
+if gitConfigBool(core.ignorecase):
+res['depotFile'] = res['depotFile'].lower()
self.client_spec_path_cache[res['depotFile']] =
self.convert_client_path(res[clientFile])
# not found files or unmap files set to
for depotFile in fileArgs:
+if gitConfigBool(core.ignorecase):
+depotFile = depotFile.lower()
if depotFile not in self.client_spec_path_cache:
self.client_spec_path_cache[depotFile] =
@@ -1962,6 +1966,9 @@ class View(object):
depot file should live. Returns if the file should
not be mapped in the client.
+if gitConfigBool(core.ignorecase):
+depot_path = depot_path.lower()
+
if depot_path in self.client_spec_path_cache:
return self.client_spec_path_cache[depot_path]
diff --git a/t/t9821-git-p4-path-variations.sh
b/t/t9821-git-p4-path-variations.sh
new file mode 100755
index 000..81e46ac
--- /dev/null
+++ b/t/t9821-git-p4-path-variations.sh
@@ -0,0 +1,200 @@
+#!/bin/sh
+
+test_description='Clone repositories with path case variations'
+
+. ./lib-git-p4.sh
+
+test_expect_success 'start p4d with case folding enabled' '
+ start_p4d -C1
+'
+
+test_expect_success 'Create a repo with path case variations' '
+ client_view //depot/... //client/...
+ (
+ cd $cli
+
+ mkdir -p Path/to
+ Path/to/File2.txt
+ p4 add Path/to/File2.txt
+ p4 submit -d Add file2
+ rm -rf Path
+
+ mkdir -p path/TO
+ path/TO/file1.txt
+ p4 add path/TO/file1.txt
+ p4 submit -d Add file1
+ rm -rf path
+
+ mkdir -p path/to
+ path/to/file3.txt
+ p4 add path/to/file3.txt
+ p4 submit -d Add file3
+ rm -rf path
+
+ mkdir -p x-outside-spec
+ x-outside-spec/file4.txt
+ p4 add x-outside-spec/file4.txt
+ p4 submit -d Add file4
+ rm -rf x-outside-spec
+ )
+'
+
+test_expect_success 'Clone root' '
+ client_view //depot/... //client/...
+ test_when_finished cleanup_git
+ (
+ cd $git
+ git init .
+ git config core.ignorecase false
+ git p4 clone --use-client-spec --destination=$git //depot
+ # This method is used instead of test -f to ensure the case is
+ # checked even if the test is executed on case-insensitive file
systems.
+ # All files are there as expected although the path cases look
random.
+ cat expect -\EOF
+ Path/to/File2.txt
+ path/TO/file1.txt
+ path/to/file3.txt
+ x-outside-spec/file4.txt
+ EOF
+ git ls-files actual
+ test_cmp expect actual
+ )
+'
+
+test_expect_success 'Clone root (ignorecase)' '
+ client_view //depot/... //client/...
+ test_when_finished cleanup_git
+ (
+ cd $git
+ git init .
+ git config core.ignorecase true
+ git p4 clone --use-client-spec --destination=$git //depot
+ # This method is used instead of test -f to ensure the case is
+ # checked even if the test is executed on case-insensitive file
systems.
+ # All files are there as expected although the path cases look
random.
+ cat expect -\EOF
+ path/TO/File2.txt
+