apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe, Jr." <wr...@rowe-clan.net>
Subject Re: [PATCH] apr_socklen_t value on HP-UX (64-bit)
Date Mon, 03 Mar 2003 17:40:08 GMT
+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 ;-)


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).
>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"
>+       ;;
> APR_CHECK_SIZEOF_EXTENDED([#include <sys/types.h>], ssize_t, 8)
> if test "$ac_cv_sizeof_ssize_t" = "$ac_cv_sizeof_int"; then

View raw message