git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2]  Add p4 read_pipe and write_pipe wrappers
       [not found] <g829vo@ger.gmane.org>
@ 2008-08-14 22:40 ` Anand Kumria
  2008-08-14 22:40   ` [PATCH 2/2] Utilise our new p4_read_pipe and p4_write_pipe wrappers Anand Kumria
  0 siblings, 1 reply; 2+ messages in thread
From: Anand Kumria @ 2008-08-14 22:40 UTC (permalink / raw)
  To: git; +Cc: simon, torarvid, Anand Kumria

 Two additional wrappers to cover 3 places where we utilise p4
 in piped form. Found by Tor Arvid Lund.

Signed-off-by: Anand Kumria <wildfire@progsoc.org>
---
 contrib/fast-import/git-p4 |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/contrib/fast-import/git-p4 b/contrib/fast-import/git-p4
index 6c64224..3e9df70 100755
--- a/contrib/fast-import/git-p4
+++ b/contrib/fast-import/git-p4
@@ -69,6 +69,10 @@ def write_pipe(c, str):
 
     return val
 
+def p4_write_pipe(c, str):
+    real_cmd = p4_build_cmd(c)
+    return write_pipe(c, str)
+
 def read_pipe(c, ignore_error=False):
     if verbose:
         sys.stderr.write('Reading pipe: %s\n' % c)
@@ -80,6 +84,9 @@ def read_pipe(c, ignore_error=False):
 
     return val
 
+def p4_read_pipe(c, ignore_error=False):
+    real_cmd = p4_build_cmd(c)
+    return read_pipe(real_cmd, ignore_error)
 
 def read_pipe_lines(c):
     if verbose:
-- 
1.5.6.3

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [PATCH 2/2]  Utilise our new p4_read_pipe and p4_write_pipe wrappers
  2008-08-14 22:40 ` [PATCH 1/2] Add p4 read_pipe and write_pipe wrappers Anand Kumria
@ 2008-08-14 22:40   ` Anand Kumria
  0 siblings, 0 replies; 2+ messages in thread
From: Anand Kumria @ 2008-08-14 22:40 UTC (permalink / raw)
  To: git; +Cc: simon, torarvid, Anand Kumria


Signed-off-by: Anand Kumria <wildfire@progsoc.org>
---
 contrib/fast-import/git-p4 |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/contrib/fast-import/git-p4 b/contrib/fast-import/git-p4
index 3e9df70..12fa9d3 100755
--- a/contrib/fast-import/git-p4
+++ b/contrib/fast-import/git-p4
@@ -141,7 +141,7 @@ def setP4ExecBit(file, mode):
 def getP4OpenedType(file):
     # Returns the perforce file type for the given file.
 
-    result = read_pipe("p4 opened %s" % file)
+    result = p4_read_pipe("opened %s" % file)
     match = re.match(".*\((.+)\)\r?$", result)
     if match:
         return match.group(1)
@@ -681,7 +681,7 @@ class P4Submit(Command):
             submitTemplate = self.prepareLogMessage(template, logMessage)
             if os.environ.has_key("P4DIFF"):
                 del(os.environ["P4DIFF"])
-            diff = read_pipe("p4 diff -du ...")
+            diff = p4_read_pipe("diff -du ...")
 
             newdiff = ""
             for newFile in filesToAdd:
@@ -719,7 +719,7 @@ class P4Submit(Command):
             if self.isWindows:
                 submitTemplate = submitTemplate.replace("\r\n", "\n")
 
-            write_pipe("p4 submit -i", submitTemplate)
+            p4_write_pipe("submit -i", submitTemplate)
         else:
             fileName = "submit.txt"
             file = open(fileName, "w+")
-- 
1.5.6.3

^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-08-14 23:03 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <g829vo@ger.gmane.org>
2008-08-14 22:40 ` [PATCH 1/2] Add p4 read_pipe and write_pipe wrappers Anand Kumria
2008-08-14 22:40   ` [PATCH 2/2] Utilise our new p4_read_pipe and p4_write_pipe wrappers Anand Kumria

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).