Thanks Laine, that worked.
On 02/02/2012 02:46 AM, Laine Stump wrote:
On 02/01/2012 12:00 PM, Deepak C Shetty wrote:
On 02/01/2012 10:05 PM, Laine Stump wrote:
On 02/01/2012 11:01 AM, Deepak C Shetty wrote:
Requesting zheeshan to look, as this API was added recently by him,
as i see on the list...
jdenemar just pushed a fix.
Do you mean the 'bogus warning about the uninit variable' fix, it
does not fix the ld 'undefined reference' error
i just git pull'ed and did make, it still fails as below.
On 02/01/2012 08:25 PM, Deepak C Shetty wrote:
../src/.libs/libvirt_driver_qemu.a(libvirt_driver_qemu_la-qemu_process.o):
In function `qemuProcessStart':
/home/dpkshetty/work/libvirt/libvirt/src/qemu/qemu_process.c:3343:
undefined reference to `virCommandAllowCap'
../src/.libs/libvirt_driver_storage.a(libvirt_driver_storage_la-storage_backend_fs.o):
In function `virStorageBackendFileSystemVolResize':
/home/dpkshetty/work/libvirt/libvirt/src/storage/storage_backend_fs.c:1234:
undefined reference to `virStorageFileResize'
Ah, I saw the warning (which ends up being an error for me) and
stopped looking.
Both of those are because new patches for the first time are calling
functions that were previously defined, but not being called. When
those functions were initially created, they weren't added to
libvirt_private.syms, and later when they were finally used, the
reviewer (me) didn't think to look for the entry in
libvirt_private.syms because the functions already existed (and of
course make check worked for me because I don't build with loadable
modules).
I just pushed build-breaker patches for both errors.
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
