From: Simon Hausmann <hausmann@kde.org>
Date: Wed, 31 Jan 2007 19:16:26 +0000 (+0100)
Subject: Speed up the import of individual files from Perforce into git by passing the output... 
X-Git-Tag: v1.5.3-rc0~65^2^2~239
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=9466d479e27afe864d42e3d501489b033b8da83f;p=git.git

Speed up the import of individual files from Perforce into git by passing the output of "p4 print" directly to git fast-import. Also try to set the mode of the file in git correctly based on file type heuristics.

Signed-off-by: Simon Hausmann <hausmann@kde.org>
---

diff --git a/contrib/fast-import/p4-fast-export.py b/contrib/fast-import/p4-fast-export.py
index f3b5f35cb..72a4fd70a 100644
--- a/contrib/fast-import/p4-fast-export.py
+++ b/contrib/fast-import/p4-fast-export.py
@@ -5,12 +5,11 @@
 # Author: Simon Hausmann <hausmann@kde.org>
 # License: MIT <http://www.opensource.org/licenses/mit-license.php>
 #
-# TODO:  - support integrations (at least p4i)
+# TODO:
+#       - support integrations (at least p4i)
 #       - support incremental imports
 #       - create tags
 #       - instead of reading all files into a variable try to pipe from
-#       - p4 print directly to stdout. need to figure out file size somehow
-#         though.
 #       - support p4 submit (hah!)
 #       - don't hardcode the import to master
 #
@@ -92,6 +91,17 @@ def describe(change):
 def p4cat(path):
     return os.popen("p4 print -q \"%s\"" % path).read()
 
+def p4Stat(path):
+    output = os.popen("p4 fstat -Ol \"%s\"" % path).readlines()
+    fileSize = 0
+    mode = 644
+    for line in output:
+        if line.startswith("... headType x"):
+            mode = 755
+        elif line.startswith("... fileSize "):
+            fileSize = long(line[12:])
+    return mode, fileSize
+
 def stripRevision(path):
     hashPos = path.rindex("#")
     return path[:hashPos]
@@ -112,7 +122,6 @@ def getUserMap():
 
     return users
 
-
 users = getUserMap()
 
 output = os.popen("p4 changes %s...%s" % (prefix, changeRange)).readlines()
@@ -133,9 +142,6 @@ for change in changes:
     [ author, log, epoch, changedFiles, removedFiles ] = describe(change)
     sys.stderr.write("\rimporting revision %s (%s%%)" % (change, cnt * 100 / len(changes)))
     cnt = cnt + 1
-#    sys.stderr.write("%s\n" % log)
-#    sys.stderr.write("%s\n" % changedFiles)
-#    sys.stderr.write("%s\n" % removedFiles)
 
     print "commit refs/heads/master"
     if author in users:
@@ -154,10 +160,13 @@ for change in changes:
             sys.stderr.write("\nchanged files: ignoring path %s outside of %s in change %s\n" % (f, prefix, change))
             continue
         relpath = f[len(prefix):]
-        print "M 644 inline %s" % stripRevision(relpath)
-        data = p4cat(f)
-        print "data %s" % len(data)
-        sys.stdout.write(data)
+
+        [mode, fileSize] = p4Stat(f)
+
+        print "M %s inline %s" % (mode, stripRevision(relpath))
+        print "data %s" % fileSize
+        sys.stdout.flush();
+        os.system("p4 print -q \"%s\"" % f)
         print ""
 
     for f in removedFiles: