You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by br...@locus.apache.org on 2000/11/28 23:43:02 UTC

cvs commit: apr hints.m4

brane       00/11/28 14:43:01

  Modified:    .        hints.m4
  Log:
  HP-UX needs _XOPEN_SOURCE_EXTENDED to find hl_error in <netdb.h>.
  
  Revision  Changes    Path
  1.25      +3 -3      apr/hints.m4
  
  Index: hints.m4
  ===================================================================
  RCS file: /home/cvs/apr/hints.m4,v
  retrieving revision 1.24
  retrieving revision 1.25
  diff -u -r1.24 -r1.25
  --- hints.m4	2000/11/28 21:31:52	1.24
  +++ hints.m4	2000/11/28 22:43:00	1.25
  @@ -82,11 +82,11 @@
   	APR_SETIFNULL(CFLAGS, [-DHIUX])
   	;;
       *-hp-hpux11.*)
  -	APR_SETIFNULL(CFLAGS, [-DHPUX11])
  +	APR_SETIFNULL(CFLAGS, [-DHPUX11 -D_XOPEN_SOURCE_EXTENDED])
   	APR_SETIFNULL(LIBS, [-lm -lpthread])
   	;;
       *-hp-hpux10.*)
  -	APR_SETIFNULL(CFLAGS, [-DHPUX10])
  +	APR_SETIFNULL(CFLAGS, [-DHPUX10 -D_XOPEN_SOURCE_EXTENDED])
    	case $host in
    	  *-hp-hpux10.01)
   dnl	       # We know this is a problem in 10.01.
  @@ -96,7 +96,7 @@
    	esac
   	;;
       *-hp-hpux*)
  -	APR_SETIFNULL(CFLAGS, [-DHPUX])
  +	APR_SETIFNULL(CFLAGS, [-DHPUX -D_XOPEN_SOURCE_EXTENDED])
   	APR_SETIFNULL(LIBS, [-lm])
   	;;
       *-linux-*)
  
  
  

Re: cvs commit: apr hints.m4

Posted by Branko Čibej <br...@xbc.nu>.
brane@locus.apache.org wrote:

>   Log:
>   HP-UX needs _XOPEN_SOURCE_EXTENDED to find hl_error in <netdb.h>.

Nuts. That's "h_errno", not "hl_error". My apologies. Brain damage.


Re: cvs commit: apr hints.m4

Posted by Branko Čibej <br...@xbc.nu>.
Greg Stein wrote:

>> I would imagine you're right. I'll try to come up with an autoconf test 
>> for that, then.
> 
> 
> Oh, don't put yourself back. My point was more for future work in hints.m4
> or if somebody wants to spend some time trying to clean some hints out of
> there, in favor of autoconf macros.

To tell the truth, I was (sub)conciously taking the easy way out. An 
autoconf check like that would be slightly dirty: "Is h_errno defined in 
<netdb.h>? No? What if I define FOO? Still not there? Then what if ..." 
You get the picture. Certainly it can be parametrized, though.

Right now I was just about fit to burst what with always having to 
configure APR on HP-UX twice, because I forgot the define the first 
time, and being too lazy to write a script to do it for me ...

Yah. I'll fix that soon.

-- 
Brane Čibej
    home:   <br...@xbc.nu>             http://www.xbc.nu/brane/
    work:   <br...@hermes.si>   http://www.hermes-softlab.com/
     ACM:   <br...@acm.org>            http://www.acm.org/



Re: cvs commit: apr hints.m4

Posted by Greg Stein <gs...@lyra.org>.
On Wed, Nov 29, 2000 at 02:41:58AM +0100, Branko Cibej wrote:
> Greg Stein wrote:
> 
> > Note that hints.m4 is a "last resort". The hope is that everything can be
> > determined via autoconf feature tests, rather than per-platform tweaks.
> > 
> > I would imagine that it is possible to discover whether
> > _XOPEN_SOURCE_EXTENDED is need by writing an autoconf test. Doing so would
> > make it work on any platform, rather than just HP/UX.
> > 
> I would imagine you're right. I'll try to come up with an autoconf test 
> for that, then.

Oh, don't put yourself back. My point was more for future work in hints.m4
or if somebody wants to spend some time trying to clean some hints out of
there, in favor of autoconf macros.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/

Re: cvs commit: apr hints.m4

Posted by Branko Čibej <br...@xbc.nu>.
Greg Stein wrote:

> Note that hints.m4 is a "last resort". The hope is that everything can be
> determined via autoconf feature tests, rather than per-platform tweaks.
> 
> I would imagine that it is possible to discover whether
> _XOPEN_SOURCE_EXTENDED is need by writing an autoconf test. Doing so would
> make it work on any platform, rather than just HP/UX.
> 
I would imagine you're right. I'll try to come up with an autoconf test 
for that, then.

-- 
Brane Čibej
    home:   <br...@xbc.nu>             http://www.xbc.nu/brane/
    work:   <br...@hermes.si>   http://www.hermes-softlab.com/
     ACM:   <br...@acm.org>            http://www.acm.org/



Re: cvs commit: apr hints.m4

Posted by Greg Stein <gs...@lyra.org>.
Note that hints.m4 is a "last resort". The hope is that everything can be
determined via autoconf feature tests, rather than per-platform tweaks.

I would imagine that it is possible to discover whether
_XOPEN_SOURCE_EXTENDED is need by writing an autoconf test. Doing so would
make it work on any platform, rather than just HP/UX.

Cheers,
-g

On Tue, Nov 28, 2000 at 10:43:02PM -0000, brane@locus.apache.org wrote:
> brane       00/11/28 14:43:01
> 
>   Modified:    .        hints.m4
>   Log:
>   HP-UX needs _XOPEN_SOURCE_EXTENDED to find hl_error in <netdb.h>.
>   
>   Revision  Changes    Path
>   1.25      +3 -3      apr/hints.m4
>   
>   Index: hints.m4
>   ===================================================================
>   RCS file: /home/cvs/apr/hints.m4,v
>   retrieving revision 1.24
>   retrieving revision 1.25
>   diff -u -r1.24 -r1.25
>   --- hints.m4	2000/11/28 21:31:52	1.24
>   +++ hints.m4	2000/11/28 22:43:00	1.25
>   @@ -82,11 +82,11 @@
>    	APR_SETIFNULL(CFLAGS, [-DHIUX])
>    	;;
>        *-hp-hpux11.*)
>   -	APR_SETIFNULL(CFLAGS, [-DHPUX11])
>   +	APR_SETIFNULL(CFLAGS, [-DHPUX11 -D_XOPEN_SOURCE_EXTENDED])
>    	APR_SETIFNULL(LIBS, [-lm -lpthread])
>    	;;
>        *-hp-hpux10.*)
>   -	APR_SETIFNULL(CFLAGS, [-DHPUX10])
>   +	APR_SETIFNULL(CFLAGS, [-DHPUX10 -D_XOPEN_SOURCE_EXTENDED])
>     	case $host in
>     	  *-hp-hpux10.01)
>    dnl	       # We know this is a problem in 10.01.
>   @@ -96,7 +96,7 @@
>     	esac
>    	;;
>        *-hp-hpux*)
>   -	APR_SETIFNULL(CFLAGS, [-DHPUX])
>   +	APR_SETIFNULL(CFLAGS, [-DHPUX -D_XOPEN_SOURCE_EXTENDED])
>    	APR_SETIFNULL(LIBS, [-lm])
>    	;;
>        *-linux-*)
>   
>   
>   

-- 
Greg Stein, http://www.lyra.org/