[Libvir] with-openvz compile error

Hi all! I try to build-up libvirt-0.4.0 with openvz support : %configure --with-init-script=redhat --with-qemud-pid-file=/var/run/libvirt_qemud.pid \ --with-remote-file=/var/run/libvirtd.pid \ --with-xen-proxy=yes --with-openvz I`ve got the error on the following stage: gcc -DHAVE_CONFIG_H -I. -I.. -I../gnulib/lib -I../gnulib/lib -I../include -I../include -I../qemud -I/usr/include/libxml2 -DBINDIR=\"/usr/lib\" -DSBINDIR=\"/usr/sbin\" -DSYSCONF_DIR=\"/etc\" -DLOCALEBASEDIR=\"/usr/share/locale\" -DLOCAL_STATE_DIR=\"/var/lib\" -DGETTEXT_PACKAGE=\"libvirt\" -Wall -Wformat -Wformat-security -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wextra -Wshadow -Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Winline -Wredundant-decls -Wno-sign-compare -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fasynchronous-unwind-tables -DWITH_OPENVZ -DWITH_QEMU -DWITH_TEST -DWITH_REMOTE -DWITH_XEN -pipe -Wall -O2 -march=i686 -mtune=i686 -c openvz_driver.c -fPIC -DPIC -o .libs/libvirt_la-openvz_driver.o openvz_driver.c:562: error: conflicting types for 'openvzOpen' openvz_driver.c:76: error: previous declaration of 'openvzOpen' was here openvz_driver.c:716: warning: initialization from incompatible pointer type openvz_driver.c:773: warning: initialization from incompatible pointer type Unfortunately i`m not confident neither with C nor with C/C++ so can anyone educate me whats wrong with that declarating openvzOpen in openvz_driver.c? -- ALTLinux Team xmpp: solar AT solar.net.ru

Mikhail A. Pokidko wrote:
Hi all!
I try to build-up libvirt-0.4.0 with openvz support : %configure --with-init-script=redhat --with-qemud-pid-file=/var/run/libvirt_qemud.pid \ --with-remote-file=/var/run/libvirtd.pid \ --with-xen-proxy=yes --with-openvz
I`ve got the error on the following stage: gcc -DHAVE_CONFIG_H -I. -I.. -I../gnulib/lib -I../gnulib/lib -I../include -I../include -I../qemud -I/usr/include/libxml2 -DBINDIR=\"/usr/lib\" -DSBINDIR=\"/usr/sbin\" -DSYSCONF_DIR=\"/etc\" -DLOCALEBASEDIR=\"/usr/share/locale\" -DLOCAL_STATE_DIR=\"/var/lib\" -DGETTEXT_PACKAGE=\"libvirt\" -Wall -Wformat -Wformat-security -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wextra -Wshadow -Wcast-align -Wwrite-strings -Waggregate-return -Wstrict-prototypes -Winline -Wredundant-decls -Wno-sign-compare -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fasynchronous-unwind-tables -DWITH_OPENVZ -DWITH_QEMU -DWITH_TEST -DWITH_REMOTE -DWITH_XEN -pipe -Wall -O2 -march=i686 -mtune=i686 -c openvz_driver.c -fPIC -DPIC -o .libs/libvirt_la-openvz_driver.o openvz_driver.c:562: error: conflicting types for 'openvzOpen' openvz_driver.c:76: error: previous declaration of 'openvzOpen' was here openvz_driver.c:716: warning: initialization from incompatible pointer type openvz_driver.c:773: warning: initialization from incompatible pointer type
Unfortunately i`m not confident neither with C nor with C/C++ so can anyone educate me whats wrong with that declarating openvzOpen in openvz_driver.c?
You should contact Shuveb Hussain who manages this driver. In this case, it looks like you could just remove the openvzOpen prototype, since the function isn't called before being defined. Rich. -- Emerging Technologies, Red Hat - http://et.redhat.com/~rjones/ Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SL4 1TE, United Kingdom. Registered in England and Wales under Company Registration No. 03798903

On Dec 21, 2007 4:54 PM, Richard W.M. Jones wrote:
You should contact Shuveb Hussain who manages this driver.
In this case, it looks like you could just remove the openvzOpen prototype, since the function isn't called before being defined.
Hi, Richard I`ve made a little patch, honestly i`m not sure did i patch it in right way but now everything looks to work now. https://bugzilla.redhat.com/show_bug.cgi?id=426385
Rich.
-- Emerging Technologies, Red Hat - http://et.redhat.com/~rjones/ Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SL4 1TE, United Kingdom. Registered in England and Wales under Company Registration No. 03798903
-- ALTLinux Team xmpp: solar AT solar.net.ru

On Fri, Dec 21, 2007 at 07:02:15PM +0300, Mikhail A. Pokidko wrote:
On Dec 21, 2007 4:54 PM, Richard W.M. Jones wrote:
You should contact Shuveb Hussain who manages this driver.
In this case, it looks like you could just remove the openvzOpen prototype, since the function isn't called before being defined.
Hi, Richard I`ve made a little patch, honestly i`m not sure did i patch it in right way but now everything looks to work now.
THanks for the patch - this brokeness is my fault from when I changed the open method to add the auth stuff. I never tested it because openvz is turned off by default. I should really make the automated build system build OpenVZ so we catch this before release. Dan. -- |=- Red Hat, Engineering, Emerging Technologies, Boston. +1 978 392 2496 -=| |=- Perl modules: http://search.cpan.org/~danberr/ -=| |=- Projects: http://freshmeat.net/~danielpb/ -=| |=- GnuPG: 7D3B9505 F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 -=|

On Fri, Dec 21, 2007 at 07:02:15PM +0300, Mikhail A. Pokidko wrote:
On Dec 21, 2007 4:54 PM, Richard W.M. Jones wrote:
You should contact Shuveb Hussain who manages this driver.
In this case, it looks like you could just remove the openvzOpen prototype, since the function isn't called before being defined.
Hi, Richard I`ve made a little patch, honestly i`m not sure did i patch it in right way but now everything looks to work now.
Okay, patch applied and commited, thanks a lot ! Daniel -- Red Hat Virtualization group http://redhat.com/virtualization/ Daniel Veillard | virtualization library http://libvirt.org/ veillard@redhat.com | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/
participants (4)
-
Daniel P. Berrange
-
Daniel Veillard
-
Mikhail A. Pokidko
-
Richard W.M. Jones