On Mon, Jun 08, 2009 at 06:02:59PM -0400, Dave Allan wrote:
Doug Goldstein wrote:
The following patch stubs out virDomainRename() as an available API function. I'm just making sure I'm doing this right. I know there were some plans to include documentation about adding API to the website but I don't see it. The included patch does not include my example driver for qemu and test.
Just looking for a thumbs up/down whether I should continue or not and whether I'm adding it right.
Doug,
I haven't looked at your patch, so I can't give an opinion on it, but the attached patch contains the most recent version of the API documentation that I did. If you apply the patch to your development tree it'll create the directory:
libvirt/docs/api_extension
with 8 example patch files in it and a single HTML doc describing the API addition process. It's still a pretty new doc, but it should cover the basics. I'd much appreciate any feedback you have as well, and I'll incorporate it.
I'm going to commit this doc to the website shortly, with a few changes to make it pick up the correct style sheets & navigation Regards, Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|