ID: 31019
User updated by: adamg at pld-linux dot org
Reported By: adamg at pld-linux dot org
Status: Open
Bug Type: Compile Failure
Operating System: Irrelevant
PHP Version: 4.3.10RC1
New Comment:
As of 4.3.10RC1 following line was introduced in ext/mssql/config.m4
if test ! -r "$FREETDS_INSTALLATION_DIR/lib/libtds.a" || test ! -r "$F
REETDS_INSTALLATION_DIR/lib/libtds.so"; then
Which, translated into human, means:
"IF either libtds.a OR libtds.so is not readable by, fail"
Which is a nonsense, since it requires presence of both of the files -
otherwise configure script will refuse to compile.
Obviously the || should be changed to &&.
Following trivial patchfixes that issue.
--- php-4.3.10RC1/ext/mssql/config.m4~ 2004-12-08 11:52:30.205750088
+0100
+++ php-4.3.10RC1/ext/mssql/config.m4 2004-12-08 11:52:51.807466128
+0100
@@ -32,7 +32,7 @@
fi
fi
- if test ! -r "$FREETDS_INSTALLATION_DIR/lib/libtds.a" || test ! -r
"$FREETDS_INSTALLATION_DIR/lib/libtds.so"; then
+ if test ! -r "$FREETDS_INSTALLATION_DIR/lib/libtds.a" && test ! -r
"$FREETDS_INSTALLATION_DIR/lib/libtds.so"; then
AC_MSG_ERROR(Could not find
$FREETDS_INSTALLATION_DIR/lib/libtds.[a|so])
fi
Previous Comments:
------------------------------------------------------------------------
[2004-12-08 12:35:58] adamg at pld-linux dot org
Description:
------------
As of 4.3.10RC1 following line was introduced in ext/mssql/config.m4
if test ! -r "$FREETDS_INSTALLATION_DIR/lib/libtds.a" || test ! -r "$F
REETDS_INSTALLATION_DIR/lib/libtds.so"; then
Which, translated into human, means:
"IF either libtds.a OR libtds.so is not readable by, fail"
Which is a nonsense, since it requires presence of both of the files -
otherwise configure script will refuse to compile.
Obviously the || should be changed to &&.
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=31019&edit=1
