According to URL, a later version than 1.7 was out even before the initial commit of 1.7 to the tree. Sadly, the new version can use (a deprecated) libpcap and doesn't like what it finds in the current stable version: >>> Compiling source in /var/tmp/portage/net-analyzer/dhcpdump-1.8/work/dhcpdump-1.8 ... cc -O2 -pipe -march=athlon-xp -c -o dhcpdump.o dhcpdump.c pod2man --section 8 \ --date "23 June 2008" \ --name "DHCPDUMP" \ --center "User Contributed Software" \ dhcpdump.pod dhcpdump.8 In file included from dhcpdump.c:30: strsep.c:63: error: expected identifier or ‘(’ before ‘__extension__’ strsep.c:63: error: expected identifier or ‘(’ before ‘)’ token strsep.c:66: error: expected identifier or ‘(’ before ‘{’ token dhcpdump.c: In function ‘pcap_callback’: dhcpdump.c:153: warning: incompatible implicit declaration of built-in function ‘strftime’ dhcpdump.c:154: warning: passing argument 4 of ‘strftime’ makes pointer from integer without a cast dhcpdump.c:159: warning: passing argument 2 of ‘strcpy’ makes pointer from integer without a cast dhcpdump.c:161: warning: passing argument 2 of ‘strcpy’ makes pointer from integer without a cast dhcpdump.c:166: error: ‘struct udphdr’ has no member named ‘uh_ulen’ dhcpdump.c:169: error: ‘struct udphdr’ has no member named ‘uh_ulen’ make: *** [dhcpdump.o] Error 1 make: *** Waiting for unfinished jobs....
Created attachment 169874 [details, diff] files/dhcpdump-1.8-Makefile.patch Adhere to CFLAGS and LDFLAGS - doesn't solve the problem mentioned in the Description.
Created attachment 169877 [details, diff] dhcpdump-1.8-Makefile.patch Ignore strsep.c completely.
Created attachment 169880 [details, diff] dhcpdump/files/dhcpdump-1.8-udp-len.patch __FAVOR_BSD shamelessly copied from /usr/include/netinet/udp.h
Created attachment 169881 [details, diff] dhcpdump-1.8-include.patch Shut gcc up by using the proper includes.
Created attachment 169883 [details, diff] dhcpdump-1.8-ebuild.patch 1) add src_unpack() to patch stuff [and inherit eutils] 2) add src_compile to adhere to toolchain [and inherit toolchain-funcs] 3) replace the missing make install target
Created attachment 169885 [details, diff] dhcpdump-1.8-ebuild.patch 4) Add the net-libs/libpcap dependency.
Debian has more interesting changes at [1]. [1] http://ftp.de.debian.org/debian/pool/main/d/dhcpdump/dhcpdump_1.8-2.diff.gz
Created attachment 169890 [details] dhcpdump-1.8-debian.patch Debian's patch, stripped down to the dhcpdump.c changes, fixes some signedness warnings that I get on hppa: dhcpdump.c: In function 'pcap_callback': dhcpdump.c:154: warning: pointer targets in passing argument 1 of 'strftime' differ in signedness dhcpdump.c:155: warning: pointer targets in passing argument 1 of 'strlen' differ in signedness dhcpdump.c:156: warning: pointer targets in passing argument 1 of 'sprintf' differ in signedness dhcpdump.c:159: warning: pointer targets in passing argument 1 of 'strcpy' differ in signedness dhcpdump.c:161: warning: pointer targets in passing argument 1 of 'strcpy' differ in signedness dhcpdump.c:163: warning: pointer targets in passing argument 1 of 'strcpy' differ in signedness dhcpdump.c:163: warning: pointer targets in passing argument 2 of 'strcpy' differ in signedness dhcpdump.c:164: warning: pointer targets in passing argument 1 of 'strcpy' differ in signedness dhcpdump.c:164: warning: pointer targets in passing argument 2 of 'strcpy' differ in signedness dhcpdump.c: In function 'printdata': dhcpdump.c:375: warning: pointer targets in passing argument 1 of '__builtin_strncpy' differ in signedness dhcpdump.c:375: warning: pointer targets in passing argument 2 of '__builtin_strncpy' differ in signedness dhcpdump.c:521: warning: pointer targets in passing argument 1 of '__builtin_strncpy' differ in signedness dhcpdump.c:521: warning: pointer targets in passing argument 2 of '__builtin_strncpy' differ in signedness However, that doesn't get rid of a runtime problem I see on hppa: elmer ~ # dhcpdump -i eth0 Ignored non IPv4 packet: 2048 Ignored non IPv4 packet: 2048 An x86 system shows the same DHCP traffic fine.
Created attachment 169892 [details, diff] dhcpdump-1.8-ebuild.patch This seems to work better.
Comment on attachment 169881 [details, diff] dhcpdump-1.8-include.patch already part of the debian patch
Comment on attachment 169880 [details, diff] dhcpdump/files/dhcpdump-1.8-udp-len.patch already part of the debian patch
its in cvs. thanks for your work - greatly appreciated. regarding the hppa issue, i'm at a loss...
/me goes and takes another piece of chocolate...
(In reply to comment #12) > its in cvs. > thanks for your work - greatly appreciated. > > regarding the hppa issue, i'm at a loss... I'll look into it later - it would appear to be a problem with IPv6, or perhaps an endian problem which has dhcpdump looking at the data in entirely the wrong way - a problem seen in many (overly simplistic) IP related packages. It shouldn't stop this bug from being RESOLVED as the package currently isn't keyworded for HPPA.
