While it is a good idea to centralize matching functionality I am opposed to replacing this small patch by an negate-enhancement. - The size of this patch (i12:d2) alone does not justify the effort to enhance negate by such a big feature.. - There is already the --skip-stderr parameter, the proposed patch only extends the already existing behaviour by issuing a warning. - Using negate would introduce at least one additional fork which is not helpful in a big environment using check_by_ssh very often and a lot of overhead for so little functionality.
-- Reply to this email on GitHub: https://github.com/monitoring-plugins/monitoring-plugins/pull/1301#issuecomment-66908797
