You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by fi...@apache.org on 2001/01/28 13:18:38 UTC

cvs commit: apr CHANGES configure.in

fielding    01/01/28 04:18:38

  Modified:    .        CHANGES configure.in
  Log:
  Remove --with-optim option in favor of OPTIM env variable.
  
  Revision  Changes    Path
  1.50      +4 -0      apr/CHANGES
  
  Index: CHANGES
  ===================================================================
  RCS file: /home/cvs/apr/CHANGES,v
  retrieving revision 1.49
  retrieving revision 1.50
  diff -u -r1.49 -r1.50
  --- CHANGES	2001/01/28 10:35:56	1.49
  +++ CHANGES	2001/01/28 12:18:38	1.50
  @@ -1,5 +1,9 @@
   Changes with APR b1  
   
  +  *) Replace configure --with-optim option by using the environment
  +     variable OPTIM instead.  This is needed because configure options
  +     do not support multiple flags separated by spaces.  [Roy Fielding]
  +
     *) Eliminate the APR_SIG* aliases for standard signal names,
        since they serve no useful purpose.  [Roy Fielding]
   
  
  
  
  1.220     +0 -2      apr/configure.in
  
  Index: configure.in
  ===================================================================
  RCS file: /home/cvs/apr/configure.in,v
  retrieving revision 1.219
  retrieving revision 1.220
  diff -u -r1.219 -r1.220
  --- configure.in	2001/01/28 11:42:31	1.219
  +++ configure.in	2001/01/28 12:18:38	1.220
  @@ -69,8 +69,6 @@
   nl='
   '
   echo $ac_n "${nl}Check for compiler flags..."
  -AC_ARG_WITH(optim,[  --with-optim="FLAGS"      compiler optimisation flags],
  -        [OPTIM="$withval"])
   
   AC_ARG_WITH(debug,[  --with-debug            Turn on debugging and compile time warnings],
           [if test "$GCC" = "yes"; then CFLAGS="$CFLAGS -g -Wall"; else CFLAGS="$CFLAGS -g"; fi])
  
  
  

Re: cvs commit: apr CHANGES configure.in

Posted by "Roy T. Fielding" <fi...@ebuilt.com>.
> Sure it can support multiple flags. If I do:
> 
> ./configure --with-optim="-O2 -O2"
> 
> It will work...

Hmmm... it wasn't working for me -- something was stripping the
double-quotes off the comand-line arguments, such that the
command-line could not be reconstructed faithfully, and thus
config.nice was getting invalid arguments.  I tried to fix that
in the macro that created config.nice, but the quotes were
being stripped earlier somewhere (I assume the option reader).

This may have been a problem in the httpd configure only, but
consensus on the list was that we should be using OPTIM anyway.

....Roy

Re: cvs commit: apr CHANGES configure.in

Posted by Dale Ghent <da...@elemental.org>.
On 28 Jan 2001 fielding@apache.org wrote:

|   +  *) Replace configure --with-optim option by using the environment
|   +     variable OPTIM instead.  This is needed because configure options
|   +     do not support multiple flags separated by spaces.  [Roy Fielding]
|   +

Sure it can support multiple flags. If I do:

./configure --with-optim="-O2 -O2"

It will work...

make[4]: Entering directory
`/local/src/apache-dev/httpd-2.0/srclib/apr/lib'
/bin/sh /local/src/apache-dev/httpd-2.0/srclib/apr/libtool --mode=compile
--silent gcc -DSOLARIS2=8 -pthreads -pthreads  -D_POSIX_PTHREAD_SEMANTICS
-D_REENTRANT -O2 -O2 -I../include -I../include/arch/unix
-I../include/arch/unix -c apr_pools.c && touch apr_pools.lo

/dale