Bug 256656

Summary: net-mgmt/icingaweb2: Fix with php 8.0
Product: Ports & Packages Reporter: Guido Falsi <madpilot>
Component: Individual Port(s)Assignee: Lars Engels <lme>
Status: Closed FIXED    
Severity: Affects Only Me CC: lme
Priority: --- Flags: bugzilla: maintainer-feedback? (lme)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch none

Description Guido Falsi freebsd_committer freebsd_triage 2021-06-16 19:36:46 UTC 
Created attachment 225870 [details]
patch

Hi,

I'm in the process of migrating some machines to php 8.0.

I noticed that icingaweb 2.8.2 does not work properly with the new PHP. The server hangs on requests in php-fpm.

Upstream performed two commits that fix this, I've created a patch to import those commits from upstream.

The relevant commits are:

https://github.com/Icinga/icingaweb2/commit/4bc5350ebaaeb7a60bbe20829a1d72def343a921

https://github.com/Icinga/icingaweb2/commit/dc7a8c8d8b6eefe03a5dd66425132de3f1afdc59

I named the new patches after the commit hashes due to lack of a better name.

With these patches it works fine with php 8.0. Also note that upstream has committed further improvements in relation to php 8.0, but these two are sufficient to get proper functionality.

I think this should be added to the official tree.
Comment 1 Lars Engels freebsd_committer freebsd_triage 2021-06-18 11:11:18 UTC 
Looks good to me. Do you like to commit it yourself?
Comment 2 Guido Falsi freebsd_committer freebsd_triage 2021-06-18 13:58:41 UTC 
Thanks, I'll commit it shortly!
Comment 3 commit-hook freebsd_committer freebsd_triage 2021-06-18 14:46:34 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=c7ba3d9534456e7c2b18811765e79cb1cad31082

commit c7ba3d9534456e7c2b18811765e79cb1cad31082
Author:     Guido Falsi <madpilot@FreeBSD.org>
AuthorDate: 2021-06-18 14:42:32 +0000
Commit:     Guido Falsi <madpilot@FreeBSD.org>
CommitDate: 2021-06-18 14:45:53 +0000

    net-mgmt/icingaweb2: Fix at runtime with php 8

    PR:             256656
    Approved by:    lme (maintainer)
    Obtained from:  https://github.com/Icinga/icingaweb2/commit/4bc5350ebaaeb7a60bbe20829a1d72def343a921
                    https://github.com/Icinga/icingaweb2/commit/dc7a8c8d8b6eefe03a5dd66425132de3f1afdc59

 net-mgmt/icingaweb2/Makefile                       |    1 +
 net-mgmt/icingaweb2/files/patch-4bc5350ebaae (new) | 1054 ++++++++++++++++++++
 net-mgmt/icingaweb2/files/patch-dc7a8c8d8b6e (new) |  193 ++++
 3 files changed, 1248 insertions(+)
Comment 4 Guido Falsi freebsd_committer freebsd_triage 2021-06-18 14:47:28 UTC 
Patch committed.

Thanks!