Bug 715527 - libvirt exists compile error in libvirt-0.9.2-1.el6.src.rpm
Summary: libvirt exists compile error in libvirt-0.9.2-1.el6.src.rpm
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: libvirt
Version: 6.2
Hardware: x86_64
OS: Linux
high
high
Target Milestone: rc
: ---
Assignee: Libvirt Maintainers
QA Contact: Virtualization Bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-06-23 08:13 UTC by Alex Jia
Modified: 2011-06-23 10:41 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-06-23 10:41:01 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Alex Jia 2011-06-23 08:13:11 UTC
Description of problem:
This issue is introduced by libvirt-Support-seemless-migration-of-SPICE-graphics-clients.patch, the function qemuMonitorJSONMigrateCancel is using variable 'type', 'hostname', 'port', 'tlsPort' and 'tlsSubject', however, these variables haven't been declared before usage, so will meet compile error when run 'make'.

Version-Release number of selected component (if applicable):
libvirt-0.9.2-1.el6.src.rpm

How reproducible:
always

Steps to Reproduce:
1. rpm -ivh http://download.devel.redhat.com/brewroot/packages/libvirt/0.9.2/1.el6/src/libvirt-0.9.2-1.el6.src.rpm
2. /root/rpmbuild/SOURCES/libvirt-0.9.2
3. apply all of pathes under the /root/rpmbuild/SOURCES dir
4. compile libvirt source code
  
Actual results:
CC     libvirt_driver_qemu_la-qemu_monitor_json.lo
qemu/qemu_monitor_json.c: In function 'qemuMonitorJSONMigrateCancel':
qemu/qemu_monitor_json.c:1946: error: 'type' undeclared (first use in this function)
qemu/qemu_monitor_json.c:1946: error: (Each undeclared identifier is reported only once
qemu/qemu_monitor_json.c:1946: error: for each function it appears in.)
qemu/qemu_monitor_json.c:1953: error: 'hostname' undeclared (first use in this function)
qemu/qemu_monitor_json.c:1954: error: 'port' undeclared (first use in this function)
qemu/qemu_monitor_json.c:1955: error: 'tlsPort' undeclared (first use in this function)
qemu/qemu_monitor_json.c:1956: error: 'tlsSubject' undeclared (first use in this function)


Expected results:
Fix it.

Additional info:
I try to apply a patch to fix this issue, it's okay to add the corresponding variables declaration for src/qemu/qemu_monitor.h,
src/qemu/qemu_monitor.c, src/qemu/qemu_monitor_json.h and src/qemu/qemu_monitor_json.c, however, I find the qemuMonitorMigrateCancel function is called by qemuMigrationProcessJobSignals function in src/qemu/qemu_migration.c,
that means I need to modify qemuMigrationProcessJobSignals function interface,
meanwhile, I need also to update other function interface such as qemuMigrationWaitForCompletion, it's not good idea.

Comment 1 Alex Jia 2011-06-23 10:41:01 UTC
The root reason is patches applied sequence issue, I haven't followed patches
sequence in SPECS/libvirt.spec, hence I met the above issue, the normal patches
sequence look like this:

# Patches
Patch1: libvirt-Turn-on-JSON-mode-and-netdev-usage-for-RHEL6-binary.patch
Patch2: libvirt-Support-virtio-disk-hotplug-in-JSON-mode.patch
Patch3: libvirt-Support-password-expiry-in-the-QEMU-driver.patch
Patch4: libvirt-Support-seemless-migration-of-SPICE-graphics-clients.patch
Patch5:
libvirt-Emit-graphics-events-when-a-SPICE-client-connects-disconnects.patch
Patch6:
libvirt-Switch-to-private-redhat-namespace-for-QMP-I-O-error-reason.patch
Patch7: libvirt-Disable-KSM-on-domain-startup.patch

Changing the bug to NOTABUG.


Alex


Note You need to log in before you can comment on or make changes to this bug.