Skip to content
This repository has been archived by the owner on Apr 12, 2019. It is now read-only.

Adds return values to the startup function #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

XCompWiz
Copy link

@XCompWiz XCompWiz commented Dec 5, 2014

The start function used to have return values to indicate success, which is used further down to control the start up of the monitor. Presently, I don't think that the monitor is started due to these returns being missing.

The start function used to have return values to indicate success, which is used further down to control the start up of the monitor. Presently, I don't think that the monitor is started due to these returns being missing.
@jeffreykog
Copy link

The monitor is not starting because the -z check in mc_startmonitor is incorrect. It should be -n.

-z checks if the length of the given string is zero, thus meaning that if $CHECKSERVER is not defined, the monitor will start.
-n checks if the length of the given string is non-zero, which is what it should be.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants