Bug #50345 [ReO->Csd]: nanosleep not detected properly on some solaris versions

From: Date: Tue, 05 Oct 2010 13:44:45 +0000
Subject: Bug #50345 [ReO->Csd]: nanosleep not detected properly on some solaris versions
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-154926@lists.php.net to get a copy of this message
Edit report at http://bugs.php.net/bug.php?id=50345&edit=1

 ID:                 50345
 Updated by:         tony2001@php.net
 Reported by:        sle at ocf dot berkeley dot edu
 Summary:            nanosleep not detected properly on some solaris
                     versions
-Status:             Re-Opened
+Status:             Closed
 Type:               Bug
 Package:            Compile Failure
 Operating System:   Solaris 10
 PHP Version:        5.3.1
 Assigned To:        tony2001
 Block user comment: N

 New Comment:

Yep, that was a mistake (typo?) on my side, sorry.
Fix committed, thanks a lot.


Previous Comments:
------------------------------------------------------------------------
[2010-10-05 15:43:37] tony2001@php.net

Automatic comment from SVN on behalf of tony2001
Revision: http://svn.php.net/viewvc/?view=revision&revision=304088
Log: fix bug #50345 (nanosleep not detected properly on some solaris
versions)

------------------------------------------------------------------------
[2010-10-05 15:27:50] uw@php.net

This would fix it, could someone double-check and apply the patch?

nixnutz@linux-fuxh:~/php/php-src/branches/PHP_5_3> svn diff
configure.in
Index: configure.in
===================================================================
--- configure.in        (Revision 304085)
+++ configure.in        (Arbeitskopie)
@@ -611,7 +611,7 @@
 )

 dnl Some systems (like OpenSolaris) do not have nanosleep in libc
-PHP_CHECK_FUNC(nanosleep, rt)
+PHP_CHECK_FUNC_LIB(nanosleep, rt)

 dnl Check for getaddrinfo, should be a better way, but...
 dnl Also check for working getaddrinfo

------------------------------------------------------------------------
[2010-10-04 13:28:13] uw@php.net

According to previous bug comments this commit should be reverted:

nixnutz@linux-fuxh:~/php/php-src/branches/PHP_5_3> svn diff
-r297960:297961
Index: configure.in
===================================================================
--- configure.in        (Revision 297960)
+++ configure.in        (Revision 297961)
@@ -611,7 +611,7 @@
 )

 svn log -r297961
------------------------------------------------------------------------
r297961 | tony2001 | 2010-04-13 14:41:46 +0200 (Di, 13. Apr 2010) | 2
Zeilen

simplify configure check for nanosleep()

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


 dnl Some systems (like OpenSolaris) do not have nanosleep in libc
-AC_CACHE_CHECK([for nanosleep], ac_cv_func_nanosleep,
[PHP_CHECK_FUNC_LIB(nanosleep, rt)])
+PHP_CHECK_FUNC(nanosleep, rt)

 dnl Check for getaddrinfo, should be a better way, but...
 dnl Also check for working getaddrinfo


Antony, can you shed some light on it? Thanks!

------------------------------------------------------------------------
[2010-10-04 13:19:32] uw@php.net

Manually setting -lrt will fix it, but maybe this could be fixed in the
source? This breaks Solaris 8, 9 and 10 builds. Any PHP version since at
least 5.3.1 is affected.



NAME
     nanosleep - high resolution sleep

SYNOPSIS
     cc [ flag... ] file... -lrt [ library... ]
     #include <time.h>

     int nanosleep(const struct timespec *rqtp,  struct  timespec
     *rmtp);

------------------------------------------------------------------------
[2010-09-17 19:42:57] svinopas at gmail dot com

I am experiencing this issue on Solaris 09/10 SPARC and 5.3.3.
Is there fix available yet?

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


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    http://bugs.php.net/bug.php?id=50345


-- 
Edit this bug report at http://bugs.php.net/bug.php?id=50345&edit=1


Thread (21 messages)

« previous php.bugs (#154926) next »