Re: [libvirt] [PATCH] (not for inclusion) add virDomainRename API

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.