Opened 12 years ago

Closed 12 years ago

#32523 closed defect (fixed)

libvirt: update to 0.9.8

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: jclift@…
Priority: Normal Milestone:
Component: ports Version: 2.0.3
Keywords: haspatch Cc:
Port: libvirt

Description

0.9.8 appears to be the latest version available. This patch makes the following changes to the port:

  • update to 0.9.8
  • fix license, based on headers of many of the source files
  • remove the block that disabled yajl when building universal, for two reasons: 1) yajl builds fine universal and libvirt builds fine universal with yajl; 2) removing the configure arg "--with-yajl" did not in fact result in yajl support being disabled if it was already installed
  • remove configure arg "--without-one" because the configure script says "WARNING: unrecognized options: --without-one". The ChangeLog shows the Open Nebula driver was removed 2011-03-28.
  • change configure arg "--with-openvz" to "--without-openvz" because the configure script says "error: The OpenVZ driver can be enabled on Linux only."

I don't know if it works, and I did not read through the NEWS or ChangeLog files (because the lists of changes since 0.8.8 are thousands and tens-of-thousands of lines long, respectively) but it does compile for me on Snow Leopard with Xcode 3.2.6.

Attachments (1)

libvirt-0.9.8.diff (1.9 KB) - added by ryandesign (Ryan Carsten Schmidt) 12 years ago.
proposed patch

Download all attachments as: .zip

Change History (5)

Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: libvirt-0.9.8.diff added

proposed patch

comment:1 Changed 12 years ago by jclift@…

This is cool. Do you need anything from me?

comment:2 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Just to say that you approve the changes, then I can commit them. Or if you have any other changes you want made let me know or attach them here.

comment:3 Changed 12 years ago by jclift@…

The patched Portfile seems good. It compiles libvirt 0.9.8 fine, and virsh appears to work ok (in limited testing so far).

So, yep. Definitely approve the changes.

Thanks Ryan. :)

comment:4 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.