Re: [PATCH 1/4] vcpupin: improve vcpupin definition of virsh vcpupin

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

 



On Mon, 20 Jun 2011 12:32:50 +0100
"Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote:
 
> Unfortunately this does not appear to build cleanly, due to const-cast warnings:
> 
> make[3]: Entering directory `/home/berrange/src/virt/libvirt/tools'
>   CC     virsh-virsh.o
> cc1: warnings being treated as errors
> virsh.c: In function 'cmdVcpupin':
> virsh.c:3064:9: error: assignment discards qualifiers from pointer target type
> virsh.c:3084:9: error: passing argument 1 of 'virParseNumber' from incompatible pointer type
> ../src/util/util.h:172:5: note: expected 'const char **' but argument is of type 'char **'
> virsh.c:3092:9: error: passing argument 1 of 'virSkipSpaces' from incompatible pointer type
> ../src/util/util.h:171:6: note: expected 'const char **' but argument is of type 'char **'
> virsh.c:3106:13: error: passing argument 1 of 'virSkipSpaces' from incompatible pointer type
> ../src/util/util.h:171:6: note: expected 'const char **' but argument is of type 'char **'
> virsh.c:3108:13: error: passing argument 1 of 'virParseNumber' from incompatible pointer type
> ../src/util/util.h:172:5: note: expected 'const char **' but argument is of type 'char **'
> virsh.c:3119:13: error: passing argument 1 of 'virSkipSpaces' from incompatible pointer type
> ../src/util/util.h:171:6: note: expected 'const char **' but argument is of type 'char **'
> virsh.c:3124:13: error: passing argument 1 of 'virSkipSpaces' from incompatible pointer type
> ../src/util/util.h:171:6: note: expected 'const char **' but argument is of type 'char **'
> make[3]: *** [virsh-virsh.o] Error 1
> 
> A reminder to everyone who commits. It is essential to run configure +
> build with  --enable-compile-warnings=error before pushing patches

 Sorry, and thank you for fixing.

-- 
Taku Izumi <izumi.taku@xxxxxxxxxxxxxx>

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