On 07/26/2012 08:58 AM, Eric Blake wrote:
On 07/25/2012 10:04 PM, Osier Yang wrote:
> On 2012年07月26日 00:02, Osier Yang wrote:
>> There were patches on virsh committed since this set created. I run
>> out of time today to resolve the conflicts to push the set. Please
>> wait a while if you have any changes on virsh (to avoid I could
>> destroy your changes with the flood when rebasing, and as a trade,
>> I would save a bit time, :-)
>
> Pushed the set with various nits fixed.
I didn't notice this beforehand, but why do we have virsh.c but
virt-helper.c? Should all the files be named virsh-helper.c?
My apologies - I'm confusing the source files for virt-host-validate
with the files for virsh. I guess that means I'm now wondering if we
should refactor tools into having subdirectories, one per tool?
--
Eric Blake eblake(a)redhat.com +1-919-301-3266
Libvirt virtualization library
http://libvirt.org