Re: [PATCH] cman/init.d/cman.in: don't assume chkconfig exists

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Guido,

in future, can you please send patches to cluster-devel@xxxxxxxxxx? It´s
easier for me to spot them.

On 3/10/2010 10:27 PM, Guido Günther wrote:
> Hi,
> attached patch makes sure we don't rely on chkconffig (which doesn't
> exist on Debian based distros). It also checks additionally for
> network-manager since this is the name of the service on Debian/Ubuntu.
> Cheers,
>  -- Guido

I am not entirely sure why we need this patch.

> # deb based distros
> if [ -d /etc/default ]; then
>         [ -f /etc/default/cluster ] && . /etc/default/cluster
>         [ -f /etc/default/cman ] && . /etc/default/cman
>         [ -z "$LOCK_FILE" ] && LOCK_FILE="/var/lock/cman"
>         type chkconfig > /dev/null 2>&1 || alias chkconfig=local_chkconfig
> fi

local_chkconfig mimics chkconfig behavior on Debian based systems that
don´t have chkconfig.

It was tested successfully on Ubuntu, but I don´t see why it would work
on Debian. If the local_chkconfig is broken, then we need to fix that as
it is used also for xen_bridge workaround.

Thanks
Fabio

--
Linux-cluster mailing list
Linux-cluster@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/linux-cluster


[Index of Archives]     [Corosync Cluster Engine]     [GFS]     [Linux Virtualization]     [Centos Virtualization]     [Centos]     [Linux RAID]     [Fedora Users]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite Camping]

  Powered by Linux