Re: [libvirt] [PATCH] fix MinGW compilation(200808)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 06, 2008 at 11:52:20AM +0200, Jim Meyering wrote:
> "Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote:
> >> 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.
> 
> Ok.  Here's the complete patch, including the new file from gnulib.
> I've verified that a freshly-cloned directory passes
> 
>   ./autogen && make && make check
> 
> Ha!  But it failed "make syntax-check" due to the
> use of ctype's tolower in inet_pton.c.
> I'm fixing that in gnulib now, since it can/should be c_tolower.
> Here's the fix prior to the s/tolower/c_tolower/ inet_pton.c change
> (it also does s/<ctype/<c-ctype/).  I'll wait until gnulib
> is fixed before committing here.

  +1 fine by me

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard@xxxxxxxxxx  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]