check_flexlm has several design flaws (#1387)

waja notifications at github.com
Wed Oct 21 17:38:55 CEST 2015


that this patch partially addresses (addresses points 2 to 4):
1/ it requires the PATH_TO_LMSTAT variable to be setup in util.pm while this
file is nnot a config file. Thus each time the package is upgraded, the
config get lost
2/ it doesn't support finding lmstat in PATH
3/ it doesn't support getting lmstat binary path from command line switchs
4/ usage is wrongly documented (needs the -F switch to provide the licence
file PATH)
5/ LM_LICENCE_FILE variable not supported
6/ licence directory instead of licence path not supported.

The patch was proposed at
https://github.com/nagios-plugins/nagios-plugins/issues/118 and I adjusted
it to apply it to our code.
You can view, comment on, or merge this pull request online at:

  https://github.com/monitoring-plugins/monitoring-plugins/pull/1387

-- Commit Summary --

  * check_flexlm has several design flaws

-- File Changes --

    M plugins-scripts/check_flexlm.pl (21)

-- Patch Links --

https://github.com/monitoring-plugins/monitoring-plugins/pull/1387.patch
https://github.com/monitoring-plugins/monitoring-plugins/pull/1387.diff

-- 
Reply to this email on GitHub:
https://github.com/monitoring-plugins/monitoring-plugins/pull/1387
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.monitoring-plugins.org/archive/devel/attachments/20151021/f0cef41b/attachment.html>


More information about the Devel mailing list