For the record, I don't mean to be directing these emails personally at* *Faré. They are for whomever the spirit moves on the asdf-devel list (i've changed Faré to be a CC here).
With my patch applied, the only failure now has to do with "awkward legacy output capture via run-shell-command."
As far as I'm concerned that test can be retired at some point anyway (personally I'm only interested in uiop:run-program), so I would be ok with leaving this as-is and moving on.
But as far as I can tell, my patch is needed (relative to current master HEAD) for
(run-program '("echo" "ok" "1"))
even to function in CCL/Win. With my patch applied, there is no failure for the
(run-program '("echo" "ok" "1"))
test -- only for the "awkward legacy output capture via run-shell-command" test.
Anyway I'll wait to hear from the maintainer regarding applying that patch.
Latest test results (again) which show that the only failure is for the awkward legacy output thingie (and yes, it has to do with an extra space printed after the "1").
https://www.dropbox.com/sh/jc2cqwpkp06dupm/G3xWZXMC6b/cl/tests/asdf/output/2...
On Thu, Sep 5, 2013 at 11:19 PM, Faré fahree@gmail.com wrote:
For the record, I'm not going to apply this patch or anything. I'll let the maintainer decide.
Also, the "ok 1 " might be a bug in CCL if CCL is the one adding a space after the last argument. I'm not going to do any debugging to confirm or infirm. I'll only actively fix bugs I introduced on platforms I can use. Good luck.
—♯ƒ • François-René ÐVB Rideau •Reflection&Cybernethics• http://fare.tunes.org Never underestimate the power of human stupidity. — Robert A. Heinlein
On Thu, Sep 5, 2013 at 11:08 PM, Genworks Support support@genworks.com wrote:
On Thu, Sep 5, 2013 at 10:49 PM, Dave Cooper david.cooper@genworks.com wrote:
Also (for my own production purposes) I made a local change to make it prepend ("cmd" "/c" ...) for Clozure/Windows when the command is given
as a
list instead of a string. I'll send that separately as a patch for your consideration. But this patch doesn't fix the one remaining test
failure;
that is something different.
Here is the patch to make it prepend ("cmd" "/c") for uiop:run-program
for
Clozure/Windows list commands, which seems to be the prudent thing to do
for
Clozure/Windows. The patch is relative to current master HEAD.
Again this does not fix the one outstanding test failure; that is still outstanding in case the spirit moves anyone to look at that before I
have a
chance to do so. Latest test results continue to be here:
https://www.dropbox.com/sh/jc2cqwpkp06dupm/G3xWZXMC6b/cl/tests/asdf/output/2...
-- My Best,
Dave Cooper, Genworks Support david.cooper@genworks.com, dave.genworks.com(skype) USA: 248-327-3253(o), 1-248-330-2979(mobile) UK: 0191 645 1699