On Wed, Mar 07, 2001 at 09:39:36PM -0800, Guy Harris wrote:
> On Wed, Mar 07, 2001 at 11:20:52PM -0600, ethereal-dev@xxxxxxxxxxxxxxxxxx wrote:
> > Build patches:
> > 1. Added --with-libpcap-include=DIR and --with-libpcap-lib=DIR
> > to specify path to libpcap library.
> > 2. Added --with-zlib=DIR to specify the path to zlib
> > 3. Remove hard-coding /usr/local/include and /usr/local/lib. This
> > is gross.
>
> 4. Cause "autogen.sh" to fail with:
>
> /usr/local/bin/autoheader: Symbol `HAVE_LIBPCAP' is not covered by
> /usr/local/share/autoconf/acconfig.h ./acconfig.h
>
> in the "wiretap" directory.
>
> One could add it to "wiretap/acconfig.h", but note that wiretap doesn't
> *use* HAVE_LIBPCAP, so the right thing to do is probably to remove the
> stuff that sets it from "wiretap/acinclude.m4".
>
> Another problem with the patched version of "wiretap/acinclude.m4" is
> that it uses AC_CHECK_HEADER to check for "pcap.h"; to quote the checkin
> comment for version 1.7 of "acinclude.m4":
>
> "autoheader" recognizes AC_CHECK_HEADERS as a macro that defines
> HAVE_XXX_H if xxx.h is present, but doesn't recognize AC_CHECK_HEADER as
> such, and thus doesn't put anything about HAVE_XXX_H into "config.h.in",
> and thus HAVE_XXX_H doesn't get defined even if xxx.h is present.
>
> Furthermore, the patched version appears to fail if "pcap.h" is not
> present - this is *not* what it is supposed to do, it's simply supposed
> to fail to define HAVE_PCAP_H, so that you can build Ethereal even if
> you don't have "pcap.h" (you can't capture packets, and you won't
> necessarily be able to read all types of libpcap capture files, but
> that's still better than a poke in the eye with a sharp stick).
>
> It also doesn't need to check for <net/bpf.h>, especially if you don't
> have "pcap.h".
>
> (The bottom line: don't just cut-and-paste the libpcap checks from
> Ethereal and paste them into the Wiretap autoconfig stuff; the Wiretap
> autoconfigure stuff doesn't need to do the same things the Ethereal
> autoconfigure stuff needs to do.)
Ok, should I resubmit against CVS?
--
albert chin (china@xxxxxxxxxxxxxxxxxx)