+1, this looks like the right solution to me.

I'd like another comment so we can commit for 0.9.2 today.
When my doxygenation bugfixing is complete, I tag _rc1.
On Friday evening during my regressions, I decided to build
the doxygen project.  Whoa boy, was that a bad idea for
trying to stay on schedule!  The scope of the changes (all
comments with some very small changes to #if blocks) is
really large, and I ran out of time this weekend to finish
reviewing all of my changes.

So those should be committed in an hour.  If nobody screams
by noon PST (about two hours from now) I will tag and announce
the candidate.

Since HP-UX/64 is borked without this patch, I'd really like to
see it sneak in ;-)

Bill

At 03:13 PM 2/21/2003, MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1) wrote:
>For some strange reason, some of the HP-UX networking api's  are unable to
>accept socklen_t as a 64-bit field when compiled in the 64-bit mode !!.
>Hence, any/all bit- wise operations on that field fails.
>
>Solution : Define apr_socklen_t as int (force it).
>
>Thanks
>-Madhu
>
>Index: configure.in
>===================================================================
>RCS file: /home/cvspublic/apr/configure.in,v
>retrieving revision 1.516
>diff -u -r1.516 configure.in
>--- configure.in        19 Feb 2003 12:48:48 -0000      1.516
>+++ configure.in        21 Feb 2003 21:08:49 -0000
>@@ -1157,6 +1157,12 @@
>     socklen_t_value="int"
> fi
> 
>+case "$host" in
>+    *hp-hpux* )
>+       socklen_t_value="int"
>+       ;;
>+esac
>+
> APR_CHECK_SIZEOF_EXTENDED([#include <sys/types.h>], ssize_t, 8)
> 
> if test "$ac_cv_sizeof_ssize_t" = "$ac_cv_sizeof_int"; then


Reply via email to