Re: [PATCH] build: fix typo in debug message

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

 



On 10/02/2012 06:39 AM, Eric Blake wrote:
> Introduced in commit 0caccb58.
> 
>   CC     libvirt_driver_qemu_impl_la-qemu_capabilities.lo
> ../../src/qemu/qemu_capabilities.c: In function 'qemuCapsInitQMP':
> ../../src/qemu/qemu_capabilities.c:2327:13: error: format '%d' expects argument of type 'int', but argument 8 has type 'const char *' [-Werror=format]
> 

>          if ((rc = virPidFileReadPath(pidfile, &pid)) < 0) {
> -            VIR_DEBUG("Failed to read pidfile %s: %d",
> +            VIR_DEBUG("Failed to read pidfile %s: %s",
>                        pidfile, virStrerror(-rc, ebuf, sizeof(ebuf)));

And in part caused by the fact that gcc 4.5 is still in quite common use
(RHEL 6.3 is still at gcc 4.4, in fact), but my sledgehammer in commit
c579d6b to silence spurious warnings from gcc 4.2 also killed useful
warnings in all but gcc 4.6 and beyond.  I'll see if I can rewrite that
virt-compile-warnings.m4 patch to be more fine-grained, and only disable
-Wformat for gcc 4.2 but not 4.5.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

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