[EMAIL PROTECTED] wrote:

Ah. I think I understand. Wouldn't the above be simpler and more obvious if
we wrote it like:

AC_DEFIN(APR_DOEXTRA, [
  CFLAGS="$CFLAGS $EXTRA_CFLAGS"
  EXTRA_CFLAGS=""
  LDFLAGS="$LDFLAGS $EXTRA_LDFLAGS"
  EXTRA_LDFLAGS=""
  LIBS="$LIBS $EXTRA_LIBS"
  EXTRA_LIBS=""
])

Presuming the above works as expected, then I'd like to change the code.


The docs really explain what is happening, and this will allow us to add
more EXTRA_ options later really cleanly.

These should be exported in APRVARS, too.

--
Brane Čibej
   home:   <[EMAIL PROTECTED]>             http://www.xbc.nu/brane/
   work:   <[EMAIL PROTECTED]>   http://www.hermes-softlab.com/
    ACM:   <[EMAIL PROTECTED]>            http://www.acm.org/




Reply via email to