Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH 3/3] cvsimport: start adding cvsps 3.x support

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:55:46

Chris Rorvick [off-list ref] writes:

[jc: please elide parts you are not responding to, leaving enough
lines to understand the context]
quoted
+    def command(self):
+        "Emit the command implied by all previous options."
+        return self.cvsps + "--fast-export " + self.opts
"--fast-export" string is missing a leading space.  With this fix and
the latest cvsps build I'm seeing 6 of 15 failures for t9650 which is
what I was getting out of the patched t9600.
Thanks; I'll amend it and push the result out, but I think we will
need to rip this part out and form the command string in a saner way
(e.g. if the path needs to have SP in it, e.g. "/Program Files/cvsps3",
the above would not work correctly anyway).
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help