Re: git: 0e8a4a0a60f1 - main - mail/roundcube: fix previous commit (add support for php 8.2, not 8.3).
Date: Wed, 18 Oct 2023 10:11:54 UTC
PHP 8.1 will reach its EOL in a month, and the default version in the ports tree will switch to 8.2. While I understand that some plugins might not be ready to use php 8.2, you can always install the packages using php 8.1 if that's the case, while using roundcube with php 8.2 wasn't possible before. With that being said, I suppose the "IGNORE_WITH_PHP=82" line in roundcube plugins have been added because roundcube wasn't supporting php 8.2, not because there was certainty that it was incompatible. So in my opinion the right question should be "Is it better to blindly remove such line in all plugins or let the maintainer do it after careful testing?" and not "Is it better to forbid using roundcude with 8.2 or enabling the plugins without testing?". Alexander Leidinger wrote: > Am 2023-10-17 16:06, schrieb Alex Dupre: > >> mail/roundcube: fix previous commit (add support for php 8.2, not >> 8.3). > > Note, there are roundcube plugins in the ports tree which don't have php > 8.2 support enabled. While I understand that those plugins may not be > validated with 8.2 by the submitter: we have now an inconsistency in the > roundcube infrastructure, and I'm not sure what is worse, enabling the > plugins without testing, or not having roundcube available with php 8.2. > > Bye, > Alexander. > -- Alex Dupre