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

[PATCH] Fix git p4 sync errors

From
MAMatt Arsenault <arsenm2@gmail.com>
Date
Oct 21, 2012, 01:59 UTC
Message-ID
<AC43C2B4-623F-4590-9F92-6CCA26645EFE@gmail.com>
>From 425e4dc6992d07aa00039c5bb8e8c76def591fd3 Mon Sep 17 00:00:00 2001
From: Matt Arsenault <arsenm2@gmail.com>
Date: Sat, 20 Oct 2012 18:48:45 -0700
Subject: [PATCH] git-p4: Fix not using -s option to describe

This solves errors in some cases when syncing renamed files. Other places where describe is used use the -s, except this one.

Signed-off-by: Matt Arsenault <arsenm2@gmail.com>
---
 git-p4.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/git-p4.py b/git-p4.py
index 882b1bb..e203508 100755
--- a/git-p4.py
+++ b/git-p4.py
@@ -2543,7 +2543,7 @@ class P4Sync(Command, P4UserMap):
     def importChanges(self, changes):
         cnt = 1
         for change in changes:
-            description = p4Cmd(["describe", str(change)])
+            description = p4Cmd(["describe", "-s", str(change)])
             self.updateOptionDict(description)
 
             if not self.silent:
-- 
1.7.12.2
Next: Junio C Hamano
Message 1 of 5 in “Fix git p4 sync errors”
  1. Fix git p4 sync errorsMatt Arsenault, Oct 21, 2012
  2. Junio C HamanoOct 21, 2012
  3. Matt ArsenaultOct 25, 2012
  4. Christian CouderOct 26, 2012
  5. Pete WyckoffOct 28, 2012

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.