You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@apr.apache.org by Jeff Trawick <tr...@gmail.com> on 2013/07/14 00:19:41 UTC

Re: svn commit: r1502807 - in /apr/apr/branches/1.4.x: ./ build/apr_network.m4

On Sat, Jul 13, 2013 at 11:54 AM, <rj...@apache.org> wrote:

> Author: rjung
> Date: Sat Jul 13 15:54:54 2013
> New Revision: 1502807
>
> URL: http://svn.apache.org/r1502807
> Log:
> Fix broken test for O_NONBLOCK inheritance.
>
> Followup to r1449570 resp. r1449569 (1.5.x)
> resp. r1449568 (trunk) which miss the variable
> declaration.
>
> Backport of r1502804 from trunk resp. r1502805
> (1.5.x).
>

need CHANGES entry to describe regression in 1.4.8


>
> Modified:
>     apr/apr/branches/1.4.x/   (props changed)
>     apr/apr/branches/1.4.x/build/apr_network.m4
>
> Propchange: apr/apr/branches/1.4.x/
>
> ------------------------------------------------------------------------------
>   Merged /apr/apr/trunk:r1502804
>
> Modified: apr/apr/branches/1.4.x/build/apr_network.m4
> URL:
> http://svn.apache.org/viewvc/apr/apr/branches/1.4.x/build/apr_network.m4?rev=1502807&r1=1502806&r2=1502807&view=diff
>
> ==============================================================================
> --- apr/apr/branches/1.4.x/build/apr_network.m4 (original)
> +++ apr/apr/branches/1.4.x/build/apr_network.m4 Sat Jul 13 15:54:54 2013
> @@ -570,6 +570,9 @@ AC_DEFUN([APR_CHECK_O_NONBLOCK_INHERITED
>  #ifdef HAVE_SYS_SOCKET_H
>  #include <sys/socket.h>
>  #endif
> +#ifdef HAVE_SYS_TIME_H
> +#include <sys/time.h>
> +#endif
>  #ifdef HAVE_SYS_SELECT_H
>  #include <sys/select.h>
>  #endif
> @@ -590,6 +593,8 @@ int main(void) {
>      int listen_port, rc;
>      struct sockaddr_in sa;
>      socklen_t sa_len;
> +    fd_set fds;
> +    struct timeval tv;
>
>      listen_s = socket(AF_INET, SOCK_STREAM, 0);
>      if (listen_s < 0) {
>
>
>


-- 
Born in Roswell... married an alien...
http://emptyhammock.com/

Re: svn commit: r1502807 - in /apr/apr/branches/1.4.x: ./ build/apr_network.m4

Posted by Rainer Jung <ra...@kippdata.de>.
On 14.07.2013 00:19, Jeff Trawick wrote:
> On Sat, Jul 13, 2013 at 11:54 AM, <rjung@apache.org
> <ma...@apache.org>> wrote:
> 
>     Author: rjung
>     Date: Sat Jul 13 15:54:54 2013
>     New Revision: 1502807
> 
>     URL: http://svn.apache.org/r1502807
>     Log:
>     Fix broken test for O_NONBLOCK inheritance.
> 
>     Followup to r1449570 resp. r1449569 (1.5.x)
>     resp. r1449568 (trunk) which miss the variable
>     declaration.
> 
>     Backport of r1502804 from trunk resp. r1502805
>     (1.5.x).
> 
> 
> need CHANGES entry to describe regression in 1.4.8

Sorry, yes, added. 0.9 and 1.3 don't need one as there has been no
release since introducing the broken fix.

Regards,

Rainer