[libvirt] [PATCH 1/3]: Cleanup the virStorageDriver structs

Daniel Veillard veillard at redhat.com
Tue Sep 23 07:43:45 UTC 2008

On Tue, Sep 23, 2008 at 07:34:17AM +0200, Chris Lalancette wrote:
> Daniel Veillard wrote:
> > On Fri, Sep 19, 2008 at 10:36:37AM +0200, Chris Lalancette wrote:
> >> This patch just cleans up the virStorageDriver structs so that they use the
> >> functionPointer = functionname syntax of gcc.  This makes it much easier to
> >> grep in the code later.  This patch also re-arranges the order of the function
> >> pointers in remote_internal so that it matches the definition of
> >> virStorageDriver in src/driver.h.  While not strictly necessary, it does make it
> >> a little more sane when looking at it in the future.
> > 
> >   Okidoc, uncontroversial, so applied and commited,
> > 
> >     thanks !
> Thanks for committing DV.  I still have to work on answering danpb's one other
> question on patch 3 (hopefully today), and then I'll post answers/updated patch
> for that one.

  Yup, note that I commited patch 3 too, the controversial point is
trivial to revert, and assuming we need different handling for pre F-9
and post F-9 having a separate patch about the issue is probably nicer

  Still I'm surprized ad the -f force option seems present on my lvm2
version on RHEL5:

test:~ -> vgremove --help 2>/dev/null | grep force
test:~ -> rpm -qif `which vgremove`
Name        : lvm2                         Relocations: (not
Version     : 2.02.32                           Vendor: Red Hat, Inc.
Release     : 4.el5                         Build Date: Fri 04 Apr 2008
01:15:19 PM CEST
Install Date: Thu 12 Jun 2008 03:15:52 PM CEST      Build Host:

  the rpm was built recently, and apparently the update to upstream was
  done around January

* Thu Jan 17 2008 Milan Broz <mbroz at redhat.com> - 2.02.30-1
- Update to latest upstream release.

  Are you sure the -f option is really the problem you're seing ?
Maybe I'm just confused, which is quite possible too :-)


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

More information about the libvir-list mailing list