>>>> "Malcolm" == Malcolm Box
<malcolm(a)brownale.demon.co.uk> writes:
Malcolm>
Malcolm> I agree that it is an invocation problem. However, it doesn't work with
Malcolm> cmd or bash, which are the only shells I have available (and one more
Malcolm> than most NT users will have!) It appears that the necessity for
Malcolm> redirection and quoting issues are mucking things up.
Can you try changing the following line in cvs-run-process:
(start-process "cvs" cvs-stderr-buffer cvs-shell "-c"
to
(start-process "cvs" cvs-stderr-buffer cvs-shell "/c"
and using /bin/sh as your cvs-shell? (This is suggested on the cyclic
site for pcl-cvs for NT). I've added a new cvs variable for this
purpose. Please let me know asap if it works.
Malcolm> Is there a good reason for using redirection rather than processing
Malcolm> everything together? From a cursory look at the code it would appear
Malcolm> possible to send the output of the cvs command to a temporary buffer,
Malcolm> then use the process sentinel to parse the entire buffer into the
Malcolm> cookie-collection. It appears this would both make the code simpler and
Malcolm> remove the need to redirect, while only modifying run-cvs-process and
Malcolm> the process sentinels. If you think this is feasible I'll try to hack
Malcolm> this up.
I was thinking about this myself, but I have to believe Per Cederqvist
had good reason for doing it that way. You might try sending him or
the guys on pcl-cvs(a)cyclic.com email before embarking down this path.
I certainly don't want to make this major a change for the XEmacs 21.0
release.
greg