Re: [PATCH] build: fix compilation of virt-login-shell.c

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

 



On Thu, Aug 08, 2013 at 02:28:53PM -0600, Eric Blake wrote:
> On 08/08/2013 02:04 PM, Jim Fehlig wrote:
> > virt-login-shell.c was failing to compile with
> > 
> > CC       virt_login_shell-virt-login-shell.o
> > virt-login-shell.c: In function 'main':
> > virt-login-shell.c:205:5: error: implicit declaration of function 'setlocale' [-Werror=implicit-function-declaration]
> > virt-login-shell.c:205:5: error: nested extern declaration of 'setlocale' [-Werror=nested-externs]
> > virt-login-shell.c:205:20: error: 'LC_ALL' undeclared (first use in this function)
> > ---
> > 
> > I'm a little surprised that others are not seeing this problem, so a bit
> > nervious committing without an ACK.
> 
> ACK - go for it.  It depends on which version of glibc you use as to
> whether you see the error (I reproduced it with glibc 2.17 of Fedora 19,
> but suspect that Dan Walsh developed the patch on an older platform).

That's strange, I compile tested this, and did the full RPM build
on my F19 host with that glibc version.

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
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]