Compile ice-9/popen.scm for MinGW

Compile ice-9/popen.scm for MinGW

[Eli Zaretskii]

Once open-process worked for me, I quickly found out that
modules/ice-9/popen.scm is not being compiled.  This is because
configure.ac explicitly excludes it for systems lacking 'fork':

   AM_CONDITIONAL([BUILD_ICE_9_POPEN],
     [test "x$enable_posix" = "xyes" && test "x$ac_cv_func_fork" = "xyes"])

Not sure how best to fix this; perhaps the test should be removed
altogether (are there any supported systems that cannot support
popen.scm?).

Re: Compile ice-9/popen.scm for MinGW

[Ludovic Courtès]

Eli Zaretskii <eliz@gnu.org> skribis:

> Once open-process worked for me, I quickly found out that
> modules/ice-9/popen.scm is not being compiled.  This is because
> configure.ac explicitly excludes it for systems lacking 'fork':
>
>    AM_CONDITIONAL([BUILD_ICE_9_POPEN],
>      [test "x$enable_posix" = "xyes" && test "x$ac_cv_func_fork" = "xyes"])
>
> Not sure how best to fix this; perhaps the test should be removed
> altogether (are there any supported systems that cannot support
> popen.scm?).

The test was in response to <http://bugs.gnu.org/13848>, which is a
MinGW issue, so it’s safe to remove it when the fork-less MinGW
replacement that you proposed is committed.

Ludo’.

Re: Compile ice-9/popen.scm for MinGW

[Eli Zaretskii]

> From: ludo@gnu.org (Ludovic Courtès)
> Cc: Mark H Weaver <mhw@netris.org>,  guile-devel@gnu.org
> Date: Mon, 30 Jun 2014 13:06:44 +0200
> 
> Eli Zaretskii <eliz@gnu.org> skribis:
> 
> > Once open-process worked for me, I quickly found out that
> > modules/ice-9/popen.scm is not being compiled.  This is because
> > configure.ac explicitly excludes it for systems lacking 'fork':
> >
> >    AM_CONDITIONAL([BUILD_ICE_9_POPEN],
> >      [test "x$enable_posix" = "xyes" && test "x$ac_cv_func_fork" = "xyes"])
> >
> > Not sure how best to fix this; perhaps the test should be removed
> > altogether (are there any supported systems that cannot support
> > popen.scm?).
> 
> The test was in response to <http://bugs.gnu.org/13848>, which is a
> MinGW issue, so it’s safe to remove it when the fork-less MinGW
> replacement that you proposed is committed.

OK, will do.  Thanks.

Re: Compile ice-9/popen.scm for MinGW

[Eli Zaretskii]

> Date: Mon, 30 Jun 2014 17:51:05 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: mhw@netris.org, guile-devel@gnu.org
> 
> > From: ludo@gnu.org (Ludovic Courtès)
> > Cc: Mark H Weaver <mhw@netris.org>,  guile-devel@gnu.org
> > Date: Mon, 30 Jun 2014 13:06:44 +0200
> > 
> > Eli Zaretskii <eliz@gnu.org> skribis:
> > 
> > > Once open-process worked for me, I quickly found out that
> > > modules/ice-9/popen.scm is not being compiled.  This is because
> > > configure.ac explicitly excludes it for systems lacking 'fork':
> > >
> > >    AM_CONDITIONAL([BUILD_ICE_9_POPEN],
> > >      [test "x$enable_posix" = "xyes" && test "x$ac_cv_func_fork" = "xyes"])
> > >
> > > Not sure how best to fix this; perhaps the test should be removed
> > > altogether (are there any supported systems that cannot support
> > > popen.scm?).
> > 
> > The test was in response to <http://bugs.gnu.org/13848>, which is a
> > MinGW issue, so it’s safe to remove it when the fork-less MinGW
> > replacement that you proposed is committed.
> 
> OK, will do.  Thanks.

Done, including the suitable changes in module/Makefile.am.