[libvirt] Remove functions library inclusion from libvirt-guests
Guido Günther
agx at sigxcpu.org
Fri Dec 17 17:56:28 UTC 2010
On Fri, Dec 17, 2010 at 07:10:27AM -0700, Eric Blake wrote:
> 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.
Something like
[ ! -f "$sysconfdir"/rc.d/init.d/functions ] || . "$sysconfdir"/rc.d/init.d/functions
would avoid a distro specific patch.
Cheers,
-- Guido
More information about the libvir-list
mailing list