diff options
author | Guy Harris <guy@alum.mit.edu> | 2017-10-14 11:55:58 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2017-10-14 18:59:43 +0000 |
commit | c2f1ab1cbcc07af13935c037534148b132aec46b (patch) | |
tree | 06f7cc3afe11324e4aff8f0053e44420888b0ba2 /acinclude.m4 | |
parent | 0943d6ee9da6d2691493600e82c6f4d938ddb3b2 (diff) |
Update comment.
Red Hat, at one point, provided a /usr/include/pcap/pcap.h without
bothering to also provide a /usr/include/pcap.h that linked to it or
included it, breaking source compatibility. That's what we're working
around when we search in both the top-level include directories and, if
they exist, pcap subdirectories of those directories.
libpcap 1.0 and later also put pcap.h in a pcap subdirectory, but also
provided a pcap.h in the top-level include directory that included
pcap/pcap.h, preserving source compatibility, so that's not the reason
we're searching in the top-level include directories and their pcap
subdirectories.
Change-Id: I8f427d46ce8293d278be9005ee623cda1ea5d691
Reviewed-on: https://code.wireshark.org/review/23902
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'acinclude.m4')
-rw-r--r-- | acinclude.m4 | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/acinclude.m4 b/acinclude.m4 index a5d9642364..9522bd6e88 100644 --- a/acinclude.m4 +++ b/acinclude.m4 @@ -195,7 +195,8 @@ AC_DEFUN([AC_WIRESHARK_PCAP_CHECK], # but we may have to look for the header in a "pcap" # subdirectory of "/usr/include" or "/usr/local/include", # as some systems apparently put "pcap.h" in a "pcap" - # subdirectory, and we also check "$prefix/include" - and + # subdirectory without also providing a "pcap.h" in the top-level + # include directory, and we also check "$prefix/include" - and # "$prefix/include/pcap", in case $prefix is set to # "/usr/include" or "/usr/local/include". # |