You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by tr...@apache.org on 2001/02/08 23:12:28 UTC

cvs commit: apr configure.in

trawick     01/02/08 14:12:28

  Modified:    .        configure.in
  Log:
  look in -lm for modf()
  
  This gets APR test programs linking on Tru64.  It may result in a
  double -lm on some platforms, as we autodetect the need for -lm
  but also hard-code the need for it in hints.m4.
  
  Revision  Changes    Path
  1.222     +1 -0      apr/configure.in
  
  Index: configure.in
  ===================================================================
  RCS file: /home/cvs/apr/configure.in,v
  retrieving revision 1.221
  retrieving revision 1.222
  diff -u -r1.221 -r1.222
  --- configure.in	2001/02/08 02:37:29	1.221
  +++ configure.in	2001/02/08 22:12:20	1.222
  @@ -226,6 +226,7 @@
   AC_SEARCH_LIBS(crypt, crypt ufc)
   AC_CHECK_LIB(truerand,main)
   AC_CHECK_LIB(iconv,iconv)
  +AC_CHECK_LIB(m,modf)
   
   dnl #----------------------------- Checks for Any required Functions
   dnl Checks for library functions. (N.B. poll is further down)
  
  
  

Re: cvs commit: apr configure.in

Posted by Greg Stein <gs...@lyra.org>.
On Thu, Feb 08, 2001 at 07:56:14PM -0500, Jeff Trawick wrote:
> Greg Stein <gs...@lyra.org> writes:
> 
> > On Thu, Feb 08, 2001 at 10:12:28PM -0000, trawick@apache.org wrote:
> > > trawick     01/02/08 14:12:28
> > > 
> > >   Modified:    .        configure.in
> > >   Log:
> > >   look in -lm for modf()
> > >   
> > >   This gets APR test programs linking on Tru64.  It may result in a
> > >   double -lm on some platforms, as we autodetect the need for -lm
> > >   but also hard-code the need for it in hints.m4.
> > 
> > Wouldn't that imply that -lm should be removed from hints.m4?
> 
> I expect that over the next few days I'll remove it for the relatively
> few platforms I have access to as I get a chance to verify.
> 
> Alternatively, we can just remove it across the board and pick up the
> pieces when somebody with one of the obscure systems gets an
> unresolved reference and we realize that we need to look for a
> different function in -lm.
> 
> Are you in favor of the latter approach?

The latter... definitely :-)

It is a bit nicer to believe we have the right solution and then apply known
fixes, then to continue to carry around baggage which we never know is still
needed or not. In this particular case, the "failure mode" is an easy
diagnosis and fix.

Ideally, hints could go away, but realistically I doubt that :-)

Cheers,
-g

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

Re: cvs commit: apr configure.in

Posted by Jeff Trawick <tr...@bellsouth.net>.
Greg Stein <gs...@lyra.org> writes:

> On Thu, Feb 08, 2001 at 10:12:28PM -0000, trawick@apache.org wrote:
> > trawick     01/02/08 14:12:28
> > 
> >   Modified:    .        configure.in
> >   Log:
> >   look in -lm for modf()
> >   
> >   This gets APR test programs linking on Tru64.  It may result in a
> >   double -lm on some platforms, as we autodetect the need for -lm
> >   but also hard-code the need for it in hints.m4.
> 
> Wouldn't that imply that -lm should be removed from hints.m4?

I expect that over the next few days I'll remove it for the relatively
few platforms I have access to as I get a chance to verify.

Alternatively, we can just remove it across the board and pick up the
pieces when somebody with one of the obscure systems gets an
unresolved reference and we realize that we need to look for a
different function in -lm.

Are you in favor of the latter approach?

Thanks,

-- 
Jeff Trawick | trawickj@bellsouth.net | PGP public key at web site:
       http://www.geocities.com/SiliconValley/Park/9289/
             Born in Roswell... married an alien...

Re: cvs commit: apr configure.in

Posted by Greg Stein <gs...@lyra.org>.
On Thu, Feb 08, 2001 at 10:12:28PM -0000, trawick@apache.org wrote:
> trawick     01/02/08 14:12:28
> 
>   Modified:    .        configure.in
>   Log:
>   look in -lm for modf()
>   
>   This gets APR test programs linking on Tru64.  It may result in a
>   double -lm on some platforms, as we autodetect the need for -lm
>   but also hard-code the need for it in hints.m4.

Wouldn't that imply that -lm should be removed from hints.m4?

Recall: hints.m4 was a bridge between the old Apache 1.3 knowledge of
systems, and our *detection* based system (embodied in autoconf).

Cheers,
-g

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