[libvirt] [PATCH 00/11] Split virsh

Eric Blake eblake at redhat.com
Mon Jul 23 23:08:52 UTC 2012


On 07/23/2012 01:29 AM, Osier Yang wrote:
> This splits virsh.c by the command groups, except 'virsh itself' group,
> each group will have a separate .c, filled with the its commands, a
> few helpers, and command group definition moved from virsh.c. and virsh.c
> simply include those new .c files.
> 
> It still could be optimized, but this set could set up the skelton, and
> further optimization could be later patches.
> 
> Osier Yang (11):
>   virsh: Move definition of cmds and cmd groups right at the top of
>     main
>   virsh: Split cmds for domain and domain monitoring from virsh.c
>   virsh: Split cmds for storage pool and storage volume from virsh.c
>   virsh: Split cmds to manage network from virsh.c
>   virsh: Split cmds to manage host interface from virsh.c
>   virsh: Split cmds to manage network filter from virsh.c
>   virsh: Split cmds to manage secret from virsh.c
>   virsh: Split cmds to manage domain snapshot from virsh.c
>   virsh: Split cmds in host group from virsh.c
>   virsh: Split cmds in node device group from virsh.c
>   virsh: Move command group definition into its own file

Patches 2 and 3 didn't come through yet (too big for the mail list, when
uncompressed), but I'm in favor of the overall idea. I'll review what I
can of the incomplete series; this has the potential to introduce
conflicts in other pending patches, but is worth doing.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120723/65f5c550/attachment-0001.sig>


More information about the libvir-list mailing list