> On my system (Ubuntu 12.10, amd64), IP_ADD_SOURCE_MEMBERSHIP is defined
> in /usr/include/x86_64-linux-gnu/bits/in.h ()
> and /usr/include/linux/in.h (). struct ip_mreq_source is
> defined in /usr/include/netinet/in.h ().
>
> On Debian GNU/Hurd, IP_ADD_SOURCE_MEMBERSHIP is defined nowhere
> in /
Am Montag, den 24.12.2012, 10:36 +1000 schrieb Ross Finlayson:
>
> On Dec 23, 2012, at 12:03 AM, Benjamin Drung
> wrote:
>
> > Am Samstag, den 22.12.2012, 13:22 +1000 schrieb Ross Finlayson:
> > > 2/ The second 'patch' seems wrong to me. I don't understand why
> > > any
> > > system would defin
Am Montag, den 24.12.2012, 10:30 +1000 schrieb Ross Finlayson:
>
> > > 1/ The first 'patch' was apparently intended to remove some
> > > compiler
> > > warnings. It's harmless, but I recommend not applying it, because
> > > the
> > > code that generates these compiler warnings might end up changi
On Dec 23, 2012, at 12:03 AM, Benjamin Drung wrote:
> Am Samstag, den 22.12.2012, 13:22 +1000 schrieb Ross Finlayson:
>> 2/ The second 'patch' seems wrong to me. I don't understand why any
>> system would define "struct ip_mreq_source", but not also define
>> "IP_ADD_SOURCE_MEMBERSHIP" (a const
>> 1/ The first 'patch' was apparently intended to remove some compiler
>> warnings. It's harmless, but I recommend not applying it, because the
>> code that generates these compiler warnings might end up changing
>> sometime (which will break the patch).
>
> Why do you cast integers/bytes into
Am Samstag, den 22.12.2012, 13:22 +1000 schrieb Ross Finlayson:
> 2/ The second 'patch' seems wrong to me. I don't understand why any
> system would define "struct ip_mreq_source", but not also define
> "IP_ADD_SOURCE_MEMBERSHIP" (a constant that makes that structure
> useful). I recommend removi
Am Samstag, den 22.12.2012, 13:22 +1000 schrieb Ross Finlayson:
> > we carry three patches (attached) in the Debian package that predate
> > my
> > involvement in the Debian packaging.
>
>
> But anyway:
> 1/ The first 'patch' was apparently intended to remove some compiler
> warnings. It's harmle
> we carry three patches (attached) in the Debian package that predate my
> involvement in the Debian packaging.
Thanks for the update. I wish the people who decided (apparently unilaterally)
that those patches were necessary had had the courtesy to check with this
mailing list first. By doing
