On Wed, Aug 06, 2008 at 09:10:04AM +0200, Jim Meyering wrote:
"Daniel P. Berrange" <berrange(a)redhat.com> wrote:
> On Tue, Aug 05, 2008 at 01:52:58PM +0200, Jim Meyering wrote:
>> Atsushi SAKAI <sakaia(a)jp.fujitsu.com> wrote:
>> ...
>> > network_conf.c:290: warning: implicit declaration of function
`inet_aton'
>> > network_conf.c:290: warning: nested extern declaration of `inet_aton'
>>
>> We can/should use inet_pton instead. Then, not only do we use what seems
>> to be the preferred interface, but there is a gnulib module by the same
>> name that can come into play if it too is missing.
>
> Even better would be to use getaddrinfo() with AI_NUMERIC, since we
> already require getaddrinfo() to work on Windows for the remote driver
Dan,
If you don't object, I'll go for the simpler interface (and smaller
change). However I've just noticed that that will require an adjustment
of the gnulib license for the inet_pton module. But that shouldn't be
a problem, since the code in question is glibc-derived.
Yep, fine by me.
Daniel
--
|: Red Hat, Engineering, London -o-
http://people.redhat.com/berrange/ :|
|:
http://libvirt.org -o-
http://virt-manager.org -o-
http://ovirt.org :|
|:
http://autobuild.org -o-
http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|