git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v2 4/8] git-p4: accommodate new move/delete type in p4

From
PWPete Wyckoff <pw@padd.com>
Date
Feb 19, 2011, 13:17 UTC
Message-ID
<1298121481-7005-5-git-send-email-pw@padd.com>
In-Reply-To
<1298121481-7005-1-git-send-email-pw@padd.com>

Change 562d53f (2010-11-21) recognized the new move/delete type for git-p4 sync, but it can also show up in an initial clone and labels output. Instead of replicating this in three places, hoist the definition somewhere global.

Signed-off-by: Pete Wyckoff <pw@padd.com>
Acked-By: Tor Arvid Lund <torarvid@gmail.com>
---
 contrib/fast-import/git-p4 |   12 +++++++-----
 1 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/contrib/fast-import/git-p4 b/contrib/fast-import/git-p4
index d2ba215..db19b17 100755
--- a/contrib/fast-import/git-p4
+++ b/contrib/fast-import/git-p4
@@ -834,6 +834,8 @@ class P4Submit(Command):
         return True
 
 class P4Sync(Command):
+    delete_actions = ( "delete", "move/delete", "purge" )
+
     def __init__(self):
         Command.__init__(self)
         self.options = [
@@ -1038,10 +1040,10 @@ class P4Sync(Command):
 
             if includeFile:
                 filesForCommit.append(f)
-                if f['action'] not in ('delete', 'move/delete', 'purge'):
-                    filesToRead.append(f)
-                else:
+                if f['action'] in self.delete_actions:
                     filesToDelete.append(f)
+                else:
+                    filesToRead.append(f)
 
         # deleted files...
         for f in filesToDelete:
@@ -1127,7 +1129,7 @@ class P4Sync(Command):
 
                 cleanedFiles = {}
                 for info in files:
-                    if info["action"] in ("delete", "purge"):
+                    if info["action"] in self.delete_actions:
                         continue
                     cleanedFiles[info["depotFile"]] = info["rev"]
 
@@ -1453,7 +1455,7 @@ class P4Sync(Command):
             if change > newestRevision:
                 newestRevision = change
 
-            if info["action"] in ("delete", "purge"):
+            if info["action"] in self.delete_actions:
                 # don't increase the file cnt, otherwise details["depotFile123"] will have gaps!
                 #fileCnt = fileCnt + 1
                 continue
-- 
1.7.4.1
Previous: Pete WyckoffNext: Junio C Hamano
Message 5 of 16 in “git-p4 fixes and enhancements”
  1. 0/8 git-p4 fixes and enhancementsPete Wyckoff, Feb 19, 2011
  2. 1/8 git-p4: test scriptPete Wyckoff, Feb 19, 2011
  3. 2/8 git-p4: fix key error for p4 problemPete Wyckoff, Feb 19, 2011
  4. 3/8 git-p4: add missing newline in initial import messagePete Wyckoff, Feb 19, 2011
  5. 4/8 git-p4: accommodate new move/delete type in p4Pete Wyckoff, Feb 19, 2011
  6. Junio C HamanoFeb 21, 2011
  7. 5/8 git-p4: reinterpret confusing p4 messagePete Wyckoff, Feb 19, 2011
  8. Tor Arvid LundFeb 23, 2011
  9. 6/8 git-p4: better message for "git-p4 sync" when not clonedPete Wyckoff, Feb 19, 2011
  10. 7/8 git-p4: decode p4 wildcard charactersPete Wyckoff, Feb 19, 2011
  11. Junio C HamanoFeb 21, 2011
  12. Pete WyckoffFeb 24, 2011
  13. Tor Arvid LundFeb 24, 2011
  14. Junio C HamanoFeb 24, 2011
  15. Pete WyckoffFeb 27, 2011
  16. 8/8 git-p4: support clone --barePete Wyckoff, Feb 19, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.