Created attachment 230515 [details] Add php81 I have prepared a patch and will be adding php81. But we need to add relevant information in Mk/Uses/php.mk to make it work. - Add php81 to possible FLAVORS of php - Move ext sodium to USE_PHP_ALL as it is common to all existing versions - Add 8.1 as Possible values in bsd.default-versions.mk
I don't seem to be able to compile a number extensions: - databases/php81-pdo_sqlite - textproc/php81-xsl - textproc/php81-xmlreader - databases/php81-pdo_dblib - lang/php81-extensions (obviously due to some dependencies) - databases/php81-pdo_mysql - databases/php81-pdo_pgsql - databases/php81-pdo_obdc - databases/php81-pdo_firebird databases/php81-pdo_pgsql says: Unknown extension pdo for PHP 81. *** Error code 1 Stop. make: stopped in /usr/ports/databases/php81-pdo_pgsql =>> Cleaning up wrkdir ===> Cleaning for php81-pdo_pgsql-8.1.1 build of databases/php81-pdo_pgsql | php81-pdo_pgsql-8.1.1 ended at Tue Jan 4 21:10:19 CET 2022 build time: 00:00:01 !!! build failure encountered !!! This is a 2022Q1 build in poudriere on 13.0-amd64.
It looks PHP 8.1 references haven't been added to Mk/Uses/php.mk or bsd.default-versions.mk yet. I'm using poudriere with a git+https ports tree and updated this morning.
(In reply to Henrik Hudson from comment #2) Yes. And this pr is specifically about that and this has been mentioned in the Commit Log. So no point in mentioning that it's not building. :) Until committed grab and use the patch which should work perfectly as it has been tested multiple times before committing php81*.
I'm sorry. I didn't even look at the commit. I assumed it was all the ports, etc.... and not just for the changes to those specific files. My bad.
I can confirm PHP81 and extensions build correctly once that patch is applied. thank you.
(In reply to Henrik Hudson from comment #5) Yeap. Hold on to the patch untill this is committed. I have some more to do to remove some compile time warnings but will take care of those after this is committed.
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=c5a743203c5c76c2b8bc427c22c2ac3bfffd37f8 commit c5a743203c5c76c2b8bc427c22c2ac3bfffd37f8 Author: Muhammad Moinur Rahman <bofh@FreeBSD.org> AuthorDate: 2022-01-10 10:45:26 +0000 Commit: Muhammad Moinur Rahman <bofh@FreeBSD.org> CommitDate: 2022-01-10 10:47:32 +0000 Welcome php81 to the tree - Move extension sodium to common as all current supported php versions in the tree supports it PR: 260774 Approved by: ale Sponsored by: Bounce Experts Mk/Uses/php.mk | 16 ++++++++++------ Mk/bsd.default-versions.mk | 2 +- 2 files changed, 11 insertions(+), 7 deletions(-)
I can confirm that php81 extensions now build fine on MAIN, however they can still not be built on the 2022Q1 branch. Can we please get that patch also applied on the quarterly branch?
(In reply to wolfgang from comment #8) At this point it's not a good idea to merge it as there is not one commit but couple of commits required to merge plus there will be lots of build on the quarterly which we don't want at this point. Sorry for this.
Thanks, I see the problem. Do you expect to merge at a later date during this quarter, or should we switch to MAIN until 2022Q2 comes around, if we need these extensions?
(In reply to wolfgang from comment #10) No we are not merging this quarter anymore. Plus we have more stuffs on PHP pending like changing to default php80 plus removing php73 which might break many things in quarter. So if php is very much important to you I would advise on sticking to main until next quarterly.