-
-
Notifications
You must be signed in to change notification settings - Fork 825
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Manual update to SimplePie 1.5.3 #2702
Conversation
Is there anything else that I can / should do to help this along? |
Hello, |
898aedb
to
bdb0389
Compare
I've updated this PR. I now started with the current files in FreshRSS's repo and manually applied the SimpliePie 1.5.2->1.5.3 changes from https://github.com/simplepie/simplepie/compare/1.5.2..1.5.3 I believe that should be much more acceptable :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good to me. Note for the other reviewers that the fix of SimplePie/Cache/File.php
was already applied in 0fb21e1
Thanks @candrews |
https://github.com/simplepie/simplepie/compare/1.5.3..a72e1dfafe7870affdae3edf0d9a494e4fa31bc6 Follow up of the update to 1.5.3 FreshRSS#2702 Requires PHP 5.6+ FreshRSS#2527 (comment)
https://github.com/simplepie/simplepie/compare/1.5.3..a72e1dfafe7870affdae3edf0d9a494e4fa31bc6 Follow up of the update to 1.5.3 #2702 Requires PHP 5.6+ #2527 (comment)
Changelogs:
I've manually applied these changes:
https://github.com/simplepie/simplepie/compare/1.5.2..1.5.3
This is of particular interest as it eliminates some warnings when run with PHP 7.4:
See simplepie/simplepie@38b5049