On Tue, 2008-09-02 at 08:50 +0200, Jim Meyering wrote:
> There, it'd be clearer to diagnose with something like
> "missing pool source device name", since there are a few
> other "name" elements.

I see Daniel V fixed this (and other) error messages to be less
ambiguous when he committed it earlier this morning.

> Not a show-stopper at all, but it'd be great if you could add a unit
> test or two, so this new code gets some regular coverage.

Are there any existing storage pool tests?  I can't find them.  I should
also add some unit tests for the storage pool discovery stuff.

--
Libvir-list mailing list
Libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to