On 24/09/2011 3:42 AM, Enrico Forestieri wrote:
On Fri, Sep 23, 2011 at 04:51:50PM +0200, Julien Rioux wrote:
I can solve the issue with the following patch. Enrico, could you
please have a look? I mostly followed what you did for Systemcall in
r38746. But I cannot test on windows.
I committed a modified version of your patch at r39742, also working on
Windows.
Thanks, your commit fixes the reported problem here as well (on ubuntu).
I had also considered directly setting the environment of forked processes,
but discarded this possibility (even if it would have been more elegant)
because execve() should have been used and thus we would have to also care
for the shebang-less scripts in texlive, needlessly complicating the code.
Sure, better to keep it simple. We do duplicate the cmd_prefix_ code in
ForkedCalls.cpp and Systemcall.cpp, but we can live with that for now.
--
Julien
