Re: HEADSUP: arp-v2 has been committed

The code in question on the Wine side is

#if defined(HAVE_SYS_SYSCTL_H) && defined(NET_RT_DUMP)
  int mib[] = {CTL_NET, PF_ROUTE, 0, AF_INET, NET_RT_FLAGS, RTF_LLINFO};

and there is nothing FreeBSD-specific in dlls/iphlpapi/ipstats.c as far
as I can see.  

If the arp-v2 update now made us incompatible both with earlier versions 
of FreeBSD and Linux, that sounds like something that should be fixed 
(instead of hacking applications like Wine).

On the other hand, the commit message at
  http://www.freebsd.org/cgi/cvsweb.cgi/src/sys/net/route.h
explicitly says
  The change in design obsoletes the semantics of RTF_CLONING, 
  RTF_WASCLONE and RTF_LLINFO routing flags. The userland applications 
  such as "arp" and "ndp" have been modified to reflect those changes.
so I guess it's not so easy.  

How many other ports are affected?

What shall we do on the Wine front?  Simply #ifdef-ing out the code in
question may not be the best of ideas, either. :-(

Gerald

On Fri, 19 Dec 2008, Vladimir Grebenschikov wrote:
> On Mon, 15 Dec 2008 06:34:13 GMT, Qing Li <qingli_at_freebsd.org> wrote:
> 
>>> The arp-v2 changes have been committed into HEAD.
>>> Please report problems to me and Kip Macy.
> 
> Wine is not build any more: 
> 
> ...
> cc -c -I. -I. -I../../include -I../../include  -D__WINESRC__  -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wwrite-strings -Wpointer-arith -I/usr/local/include -O2 -pipe -fno-strict-aliasing  -o ipstats.o ipstats.c
> ipstats.c: In function 'getNumArpEntries':
> ipstats.c:1253: error: 'RTF_LLINFO' undeclared (first use in this function)
> ipstats.c:1253: error: (Each undeclared identifier is reported only once
> ipstats.c:1253: error: for each function it appears in.)
> ipstats.c: In function 'getArpTable':
> ipstats.c:1311: error: 'RTF_LLINFO' undeclared (first use in this function)
> ipstats.c:1311: warning: initialization makes integer from pointer without a cast
> gmake[2]: *** [ipstats.o] ?????? 1
> gmake[2]: Leaving directory `/usr/ports/emulators/wine/work/wine-1.1.10/dlls/iphlpapi'
> gmake[1]: *** [iphlpapi] ?????? 2
> gmake[1]: Leaving directory `/usr/ports/emulators/wine/work/wine-1.1.10/dlls'
> gmake: *** [dlls] ?????? 2
> 
> 

-- 
Gerald (Jerry) Pfeifer   gerald_at_pfeifer.com   http://www.pfeifer.com/gerald/