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/2013-09-05/windows/ccl-1.10-f96-win-x86/ccl.text




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/2013-09-05/windows/ccl-1.10-f96-win-x86/ccl.text
>
>
> --
> 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



--
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