tcpdump mailing list archives

Re: [PATCH] Avoid losing CPPFLAGS in configure.in


From: Romain Francoise <rfrancoise () debian org>
Date: Sun, 04 Mar 2012 13:23:12 +0100

Guy Harris <guy () alum mit edu> writes:

Checked into the trunk and 4.2 branches.

Thanks, but I should have made it more explicit that the patch fixes *two*
typos: 'savedppflags' vs. 'savedcppflags' and 'CPPLAGS' vs. 'CPPFLAGS'.
You fixed only the latter, so the following is still necessary after your
commit:

diff --git a/configure.in b/configure.in
index f1300d7..8864238 100644
--- a/configure.in
+++ b/configure.in
@@ -732,7 +732,7 @@ if test $ac_cv_func_pcap_findalldevs = "yes" ; then
 dnl Check for Mac OS X, which may ship pcap.h from 0.6 but libpcap may
 dnl be 0.8; this means that lib has pcap_findalldevs but header doesn't
 dnl have pcap_if_t.
-    savedppflags="$CPPFLAGS"
+    savedcppflags="$CPPFLAGS"
     CPPFLAGS="$CPPFLAGS $V_INCLS"
     AC_CHECK_TYPES(pcap_if_t, , , [#include <pcap.h>])
     CPPFLAGS="$savedcppflags"
@@ -1067,7 +1067,7 @@ if test "$want_libcrypto" != "no"; then
                fi
                AC_CHECK_LIB(crypto, DES_cbc_encrypt)
 
-               savedppflags="$CPPFLAGS"
+               savedcppflags="$CPPFLAGS"
                CPPFLAGS="$CPPFLAGS $V_INCLS"
                AC_CHECK_HEADERS(openssl/evp.h)
                CPPFLAGS="$savedcppflags"

-- 
Romain Francoise <rfrancoise () debian org>
http://people.debian.org/~rfrancoise/
-
This is the tcpdump-workers list.
Visit https://cod.sandelman.ca/ to unsubscribe.


Current thread: