[Libvir] ruby binding patch
dlutter at redhat.com
Fri Mar 28 17:48:45 UTC 2008
thanks a lot for the patch. First off, can you send patches in the
future as text patches, either attached or inline ? I was able to
apply your hg bundle though.
On Thu, 2008-03-27 at 23:59 -0700, Vadim Zaliva wrote:
> I have attempted to add following methods to Domain class:
> 1. vcpus=
> 2. memory=
> This is my first patch to libvirt-ruby, and I will appreciate your
> My next goal is to add binding for virDomainPinVcpu() method.
The patch overall looks good, and I committed it. There were some minor
* libvirt_dom_memory_set called virDomainSetMaxMemory instead of
virDomainSetMemory (probably cut&paste error)
* there were no tests for the two new methods. In general, try to
add/expand the tests in tests/tc_connect.rb when you add to the
bindings, though that is highly dependent on what the test
driver supports. In this case though it was pretty easy.
Thanks again for doing this, and I am looking forward to the
 the hg patchbomb extension makes that very easy:
More information about the libvir-list