ID:               31019
 Updated by:       [EMAIL PROTECTED]
 Reported By:      adamg at pld-linux dot org
-Status:           Open
+Status:           Assigned
 Bug Type:         Compile Failure
 Operating System: Irrelevant
 PHP Version:      4.3.10RC1 and 5CVS
-Assigned To:      
+Assigned To:      fmk


Previous Comments:
------------------------------------------------------------------------

[2004-12-08 12:39:24] adamg at pld-linux dot org

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

------------------------------------------------------------------------

[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

Reply via email to