Re: [PATCH1/2] Use spawn() on Cygwin
Date: Fri, 2 Aug 2013 22:49:31 -0400 From: Christopher Faylor m...@cgf.cx On Tue, Jul 30, 2013 at 08:02:54PM +0300, Eli Zaretskii wrote: Date: Tue, 30 Jul 2013 11:52:58 -0500 From: Norbert Thiebaud nthieb...@gmail.com Cc: Pavel Fedin p.fe...@samsung.com, bug-make@gnu.org fork() is a very expensive operation in cygwin. Yes, I know. But without it, some things that are expected of a Posix behavior will not work. A notable example is that the child process initially has all the file descriptors and global variables that the parent had. 'spawn' does not necessarily guarantee that. Thank you Eli. This is my reservation about this change too. I maintain the make package for Cygwin and if I have reservations, you can be sure that it won't make it into the released version. And, btw, I think there's a thread on the Cygwin list just now about one such problem. ___ Bug-make mailing list Bug-make@gnu.org https://lists.gnu.org/mailman/listinfo/bug-make
Re: [PATCH1/2] Use spawn() on Cygwin
On Sat, Aug 03, 2013 at 11:16:44AM +0300, Eli Zaretskii wrote: Date: Fri, 2 Aug 2013 22:49:31 -0400 From: Christopher Faylor m...@cgf.cx On Tue, Jul 30, 2013 at 08:02:54PM +0300, Eli Zaretskii wrote: Date: Tue, 30 Jul 2013 11:52:58 -0500 From: Norbert Thiebaud nthieb...@gmail.com Cc: Pavel Fedin p.fe...@samsung.com, bug-make@gnu.org fork() is a very expensive operation in cygwin. Yes, I know. But without it, some things that are expected of a Posix behavior will not work. A notable example is that the child process initially has all the file descriptors and global variables that the parent had. 'spawn' does not necessarily guarantee that. Thank you Eli. This is my reservation about this change too. I maintain the make package for Cygwin and if I have reservations, you can be sure that it won't make it into the released version. And, btw, I think there's a thread on the Cygwin list just now about one such problem. Yep. cgf ___ Bug-make mailing list Bug-make@gnu.org https://lists.gnu.org/mailman/listinfo/bug-make
