Skip to content

Increase minimal support to PHP 5.3.8+#1618

Merged
Alkarex merged 2 commits intoFreshRSS:devfrom
Alkarex:php_5.3.8
Aug 24, 2017
Merged

Increase minimal support to PHP 5.3.8+#1618
Alkarex merged 2 commits intoFreshRSS:devfrom
Alkarex:php_5.3.8

Conversation

@Alkarex
Copy link
Copy Markdown
Member

@Alkarex Alkarex commented Aug 21, 2017

@Alkarex Alkarex added this to the 1.7.1 milestone Aug 21, 2017
@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Aug 21, 2017

PHP 5.3.6+ is needed to fix the sanitize bug of #1604
https://github.com/simplepie/simplepie/blob/45c50e895f0657480c57069b25722dfebec1c8e0/library/SimplePie/Sanitize.php#L368-L375

Patch to increase the minimal PHP version from PHP 5.3.3 to 5.3.8+, based on the fact that 5.3.6 and 5.3.7 do not seem to be used much, and we need 5.3.7+ for some crypto functions.
https://w3techs.com/technologies/details/pl-php/5.3/all (loosing <4.8% of PHP installed base)

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Aug 21, 2017

Reference to FreshRSS PHP needs: #789 (comment)

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Aug 21, 2017

@marienfressinaud Do you still use CentOS? If yes, which version?

@marienfressinaud
Copy link
Copy Markdown
Member

I'm using CentOS 7 but I plan to move quite soon to another distribution

@Alkarex Alkarex merged commit 2916637 into FreshRSS:dev Aug 24, 2017
@Alkarex Alkarex deleted the php_5.3.8 branch August 24, 2017 16:15
@Alkarex Alkarex mentioned this pull request Aug 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants