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

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