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,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia



___
Bug-make mailing list
Bug-make@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-make


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

2014-01-16 Thread Eli Zaretskii
 From: Pavel Fedin p.fe...@samsung.com
 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 earlier, nobody replied.

My take on this is that I need to see the refactoring to say something
intelligent about it.

___
Bug-make mailing list
Bug-make@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-make


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

2014-01-15 Thread Eli Zaretskii
 From: Pavel Fedin p.fe...@samsung.com
 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 rational seed in what you say. Since this code
 actually goes to generic part, it will be useful to be able to disable this
 code, just in case.
  Please take the updated patch. I have verified it to build by supplying
 CPPFLAGS=-DNO_OUTPUT_SYNC to configure.

Thanks, this variant is fine with me.

___
Bug-make mailing list
Bug-make@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-make


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,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia



___
Bug-make mailing list
Bug-make@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-make