connortechnology / ZoneMinder

ZoneMinder is a free, open source Closed-circuit television software application developed for Linux which supports IP, USB and Analog cameras.
http://www.zoneminder.com/
GNU General Public License v2.0
9 stars 9 forks source link

Provide the user an option to enable zmeventnotification #92

Closed pliablepixels closed 6 years ago

pliablepixels commented 6 years ago

There are sufficient users of zmNinja's zmeventnotitication daemon at this stage. Billions!!!!! millions!!!! hundreds of thousands!!! tens of thousands!! thousands! few hundred likely.

Problem Statement: Every time ZM is updated, the notification server gets removed from zmdc and zmpkg as its currently a manual process.

The issue with packaging the daemon is it relies on certain libraries that need some manual exploring to figure out the exact names depending on distro. I only maintain a ubuntu guide and I leave it to users to figure it out for others.

Proposed Approach: While chatting with @connortechnology on IRC, we talked about making this an optional element.

My thought is this: a) We list it as a OPT_USE_XXXX in options. We add a help text saying please install this separately from my repo manually b) We add conditional code in zmdc.pl and zmpkg.pl to add it to the lists if enabled. It is the user's responsibility to make sure the daemon is actually present.

If this is acceptable, I'll do a PR in a few days.