RE: [PATCH] Fix output-sync option on EMX - updated

2014-01-17 Thread Pavel Fedin
Hello! > Done. Thank you very much! > My take on this is that I need to see the refactoring to say something > intelligent about it. Ok, of course. :) Currently i have some high-priority problems here, when i have more time i'll get back to it and of course post the patch. Kind regards, Pa

Re: [PATCH] Fix output-sync option on EMX - updated

2014-01-16 Thread Eli Zaretskii
> From: Pavel Fedin > Cc: bug-make@gnu.org > Date: Thu, 16 Jan 2014 10:03:38 +0400 > > > Thanks, this variant is fine with me. > > Good. I'm waiting for it to be committed Done. > In order to make it switched at runtime, i would like to refactor > child_execute_job(). I have suggested this e

RE: [PATCH] Fix output-sync option on EMX - updated

2014-01-15 Thread Pavel Fedin
Hello! > Thanks, this variant is fine with me. Good. I'm waiting for it to be committed, since i would like to go on with merging spawn-patch. In order to make it switched at runtime, i would like to refactor child_execute_job(). I have suggested this earlier, nobody replied. Kind regards, Pa

Re: [PATCH] Fix output-sync option on EMX - updated

2014-01-15 Thread Eli Zaretskii
> From: Pavel Fedin > Cc: bug-make@gnu.org > Date: Wed, 15 Jan 2014 10:23:58 +0400 > > I have rechecked. Actually NO_OUTPUT_SYNC is defined only by handmade > config.h files for DOS, VMS and Amiga. EMX uses configure script, so > NO_OUTPUT_SYNC will not be defined. > However, indeed, there is ra

[PATCH] Fix output-sync option on EMX - updated

2014-01-14 Thread Pavel Fedin
Hello! > Thanks, but does EMX support output-sync? If not, this fragment: > > + /* Divert child output if output_sync in use. */ > + if (child->output.syncout) > +{ > + if (child->output.out >= 0) > +outfd = child->output.out; > + if