FS#74163 - [tt-rss] Make it compatible with extra/php7

Attached to Project: Community Packages
Opened by Chih-Hsuan Yen (yan12125) - Saturday, 19 March 2022, 04:04 GMT
Last edited by Chih-Hsuan Yen (yan12125) - Monday, 18 July 2022, 16:14 GMT
Task Type Feature Request
Category Packages
Status Closed
Assigned To Anatol Pomozov (anatolik)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
In https://bugs.archlinux.org/task/73451, tt-rss package is changed to support PHP 8.1. Could you consider changing PKGBUILD for compatibility with both PHP 8.1 and PHP 7.4? The situation with PHP 8.1 is bad for some other Arch packages (ex: nextcloud). I'd like to switch to php7 and not running both php-fpm and php7-fpm on my server for resource usage considerations.

I created a pull request (linked below) at upstream to support PHP 8.1 while keeping compatibility with 7.x. Part of the PR is accepted, while changes in vendored libraries are deferred. In the attached PKGBUILD, I updated pkgver to include merged changes, and replace `git revert` commands with the unmerged part of my PR.

Additional info:
* community/tt-rss 2:r11220.b59bde7b4-1
* Upstream discussion for PHP 8.1: https://community.tt-rss.org/t/support-for-php-8-1/5089
* My pull request for upstream: https://git-gitea.tt-rss.org/fox/tt-rss/pulls/67

Steps to reproduce:
Try to run tt-rss using php7-fpm
   PKGBUILD (2.1 KiB)
This task depends upon

Closed by  Chih-Hsuan Yen (yan12125)
Monday, 18 July 2022, 16:14 GMT
Reason for closing:  Fixed
Additional comments about closing:  tt-rss 2:r11325.8f19423c2-1
Comment by Chih-Hsuan Yen (yan12125) - Monday, 18 July 2022, 16:14 GMT
The latest upsteram version takes a similar approach as my patch. I expect updated community/tt-rss should work with extra/php7.

Loading...