On Tue, Dec 18, 2012 at 03:59:29PM -0200, Leonardo Augusto Guimarães Garcia wrote: > Hello everyone, > > I am new to virt-viewer and I've been trying to build it using the > autobuild.sh script from the git repository. However, the build > process is not working as some warnings are being hit during the > source code compilation and there is a specific flag being passed to > autogen.sh to --enable-compile-warnings=error. > > For the moment I just removed the flag and I was able to build > virt-viewer. But I was wondering whether this is the correct way of > building or if there are different instructions to build > virt-viewer. In general terms, upstream developers should enable compiler warnings, so they don't add buggy new code. Downstream packagers should disable compiler warnings because -Werror can break builds unnecessarily (extra marks for packagers who enable compiler warnings and fix the problems however!) Since you used the version from git, it's implicit that you're an upstream developer. Packagers would normally compile from the tarballs using ./configure && make. So anyhow if you don't want compiler warnings, feel free to disable that switch. But if you are planning to submit patches it's important that the new code doesn't contain any problems that could have been fixed by paying attention to the compiler warnings. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-top is 'top' for virtual machines. Tiny program with many powerful monitoring features, net stats, disk stats, logging, etc. http://et.redhat.com/~rjones/virt-top _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
