Re: [libvirt] Remove functions library inclusion from libvirt-guests
On 12/17/2010 04:13 AM, Laurent Léonard wrote:
Hi,
The attached patch removes functions library inclusion from libvirt-guests.
No function from this library is used by the script so there is no need to
include it.
NACK. It's better to follow the init script standards and blindly
include the library, even if we currently do not use functions from the
library in the rest of the script. For all you know, the mere inclusion
of the library may have some system registration side effects, where
omitting the inclusion would change the behavior of the init script.
--
Eric Blake eblake(a)redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
Attachments:
- signature.asc (application/pgp-signature — 619 bytes)