[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 1/1] Document how to extend the libvirt API



On Wed, May 27, 2009 at 01:08:04PM -0400, David Allan wrote:
> ---

  Looks good but IMHO it misses one of the main point, it focuses
only on the technical aspects of the submission but not at all on the
process and interraction with the list.
  Basically, before engaging in adding a new API for libvirt, the best
is to discuss a first draft of the suggested changes to libvirt.h,
then make sure it gets reviewed, and start developping the code only
after a first on-list validation step. There is nothing worse than
working a week on a patch sending it to the list and learning that
the thing could not work because it breaks some preestablished rules.
I think the most common example would be a patch to add raw extra
qemu command line options to the API :-)

  But that can be added next to your description, or included within
thanks a lot :-)

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]