Bug 250800 - mail/postfixadmin: requires XMLRPC and thus fails to build with PHP 8.0
Summary: mail/postfixadmin: requires XMLRPC and thus fails to build with PHP 8.0
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
Depends on:
Reported: 2020-11-02 08:56 UTC by Kamigishi Rei
Modified: 2020-11-23 23:08 UTC (History)
1 user (show)

See Also:
ports.maintainer: maintainer-feedback+

Add PHP_VER awareness to xmlrpc inclusion (888 bytes, patch)
2020-11-09 10:38 UTC, Rin Morningstar
ports.maintainer: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kamigishi Rei 2020-11-02 08:56:55 UTC
XMLRPC was moved to PECL and is no longer available in base PHP. We may need a new port, www/pecl-xmlrpc or something like that. (Note: postfixadmin's XMLRPC functionality is an optional feature and does not seem to be required for its web interface to work.)
Comment 1 Rin Morningstar 2020-11-09 10:38:07 UTC
Created attachment 219484 [details]
Add PHP_VER awareness to xmlrpc inclusion

Add a basic PHP_VER test so the port only includes XMLRPC if we're building on PHP 7.x.  PHP 8 and PFA's IPC are mutually exclusive so the real fix will happen upstream.
Comment 2 Kamigishi Rei 2020-11-09 14:18:22 UTC
Upstream issue for reference: https://github.com/postfixadmin/postfixadmin/issues/395
Comment 3 Rin Morningstar 2020-11-19 09:56:13 UTC
Have you had time to test this patch? Do you get any other regressions with PFA on PHP 8?
Comment 4 Kamigishi Rei 2020-11-19 13:17:41 UTC
(In reply to Rin Morningstar from comment #3)
I was unable to properly test the port due to various other software's incompatibilities with PHP 8.0, but it now builds the package properly for 8.0 as well.
Comment 5 Rin Morningstar 2020-11-23 23:08:11 UTC
Approved for commit.

Tested here with no regressions on php74, submitter can build test on php80.  PFA is not definitely alone in being generally unproven on php80, but this change is required to make it possible for someone to build for it.