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

[Fabio M. Di Nitto]

Hi Guido,

I haven´t seen any reply to my request for info. Did I lost emails or
missed one?

Thanks
Fabio

On 3/23/2010 7:55 AM, Fabio M. Di Nitto wrote:
> Hi Guido,
> 
> in future, can you please send patches to cluster-devel at redhat.com? 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 at redhat.com
> https://www.redhat.com/mailman/listinfo/linux-cluster