Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix install_server.sh for CentOS/RHEL #909

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
5 participants

The check to see if chkconfig exists is missing a space after !, so on CentOS/RHEL, the script will still try to use update-rc.d:

Installing service...
./install_server.sh: line 178: update-rc.d: command not found

This patch changes two occurrences (lines 162 and 176) of :

if [ !`which chkconfig` ] ; then 

to

if [ ! `which chkconfig` ] ; then 

snird commented Dec 22, 2013

This is still relevant on CentOS 6.4 with Redis 2.8.3.

spuder commented Feb 19, 2014

Still relevant on Centos 6.5 with Redis 2.8.6

icohen commented Mar 7, 2014

Still relevant on CentOS 6.5 with Redis 2.8.7

Contributor

mattsta commented Mar 7, 2014

+1 Merge. Zero risk.

@mattsta mattsta closed this May 30, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment