Code review comment for lp:~songofacandy/bzr/fix-523746-dev

Revision history for this message
Alexander Belchenko (bialix) wrote :

> So it appears that I lack the windows knowledge (and the ability to tests)
> to effectively help Naoki here.
>
> The consensus seems to be that the problem is now to either:
> - make subprocess.Popen accept unicode arguments (Naoki as a monkey-patch for
> that
> but I find it a bit scary),
> - bypass subprocess.Popen and use a direct win32 API (I'm out of my league on
> this one)
>
> John ? Alexander ? Can you give it a try ?

Sorry, I'm a bit busy these days, have lost this ping. I can't promise I will look into it soon. But I'll try.

« Back to merge proposal