Summary: | net-mgmt/nagios patch to support php7 | ||||||
---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Palle Girgensohn <girgen> | ||||
Component: | Individual Port(s) | Assignee: | Mathieu Arnold <mat> | ||||
Status: | Closed FIXED | ||||||
Severity: | Affects Only Me | Flags: | bugzilla:
maintainer-feedback?
(mat) |
||||
Priority: | --- | ||||||
Version: | Latest | ||||||
Hardware: | Any | ||||||
OS: | Any | ||||||
Attachments: |
|
Description
Palle Girgensohn
2017-12-21 20:47:02 UTC
Has this been obtained from upstream, or if you made it yourself, have you submitted it upstream? Hi! I made it myself and have not submitted upstreams. Since the fix is only for nagios-3.5, I'm not sure they will take interest? There is no branch for 3.5 in their repo so I couldn't post a pull request, only a bug report: https://github.com/NagiosEnterprises/nagioscore/issues/467 I suggest adding this trivial patch in the FreeBSD repo for now. OK to commit? Palle Here's the answer: ``` I'm not sure much will get done about this, Version 3.5 is pretty old now (more than 4 years since it was released), is there a reason why you're still using such an old version? Nagios4 already runs on PHP7, there's nothing to be done to support it. ``` I think that is reasonable. I think we should commit the patch. Perhaps also deprecate the port altogether, since it is not supported upstreams? Palle We still have the port because I tried a few times to migrate my nagios 3 installations to nagios 4, and it never worked, it was always stuck in some strange loop, or crashed, or did not run any probe. Since then I migrated to icinga, so I do not think it is needed any more. (I know, those are my needs.) Feel free to commit the patch. Yeahm nagios is a bitch sometimes. I'll commit the patch, and let's leave it at that for a while. Thanks, Palle A commit references this bug: Author: girgen Date: Tue Jan 23 22:54:04 UTC 2018 New revision: 459801 URL: https://svnweb.freebsd.org/changeset/ports/459801 Log: Help nagios cope with php-7 split has been replaced with explode PR: 224509 Changes: head/net-mgmt/nagios/Makefile head/net-mgmt/nagios/files/patch-html_includes_rss_parse.inc |