configure hangs if run in background

[GoogleCodeExporter]

The configure script does
  enabled doxygen && php -v >/dev/null 2>&1 && enable install_docs
which, oddly enough, hangs if run in the background on MacOSX 10.7
which comes with php 5.3.15.

The workaround / fix is to redirect php's stdin from /dev/null:

enabled doxygen && php -v >/dev/null 2>&1 < /dev/null && enable install_docs

Original issue reported on code.google.com by daniel.r...@gmail.com on 17 Sep 2013 at 10:15

[GoogleCodeExporter]

Here's the patch I use:

--- a/configure.x   2013-09-17 15:13:19.000000000 -0700
+++ b/configure 2013-09-17 15:13:30.000000000 -0700
@@ -162,7 +162,7 @@
 # install everything except the sources, by default. sources will have
 # to be enabled when doing dist builds, since that's no longer a common
 # case.
-enabled doxygen && php -v >/dev/null 2>&1 && enable install_docs
+enabled doxygen && php -v >/dev/null 2>&1 < /dev/null && enable install_docs
 enable install_bins
 enable install_libs

Original comment by daniel.r...@gmail.com on 25 Sep 2013 at 2:28

[GoogleCodeExporter]

Original comment by fgalli...@google.com on 17 Oct 2013 at 10:21

• Changed state: Assigned

[GoogleCodeExporter]

A patch with a potential fix has been uploaded:
https://gerrit.chromium.org/gerrit/#/c/67631/

Daniel, would you be able to test this patch to see if it fixes the issue you 
reported please? Thanks.

Original comment by agra...@google.com on 5 Nov 2013 at 9:06

[GoogleCodeExporter]

It fails here on osx 10.9 because it changed the -v to -version:

$ php -version

Parse error: parse error in Command line code on line 1

$ php -v
PHP 5.4.17 (cli) (built: Aug 25 2013 02:03:38) 
Copyright (c) 1997-2013 The PHP Group
Zend Engine v2.4.0, Copyright (c) 1998-2013 Zend Technologies

However, if you don't change the -v, the patch should work. 

Original comment by daniel.r...@gmail.com on 17 Jan 2014 at 8:19

[GoogleCodeExporter]

This patch has been abandoned (see patch 67631 commentary):
"the php test in configure has been gone since: f8630c7 configure: remove php 
test".

Original comment by agra...@google.com on 13 Aug 2014 at 9:29

• Changed state: Invalid