Update rpm packer for phpredis#1904
Conversation
|
Sorry, but I don't think it make sense to maintain a spec file here. Indeed this one is really outdated. Your proposal is really not enough (missing optional dependency, compression libraries and extensions) Better maintained in the RPM distribution itself. Especially as binary packages are also produced there and available for
IMHO, simply remove this "rpm" directory. |
|
Hey Remi, thanks for taking the time to answer me. You are making a lot of good points here. I actually did not found the rspec that you had linked here before, thank you. I was trying to understand the problem RHEL8 actually have ( As it is my need ). I found your conversation about this elsewhere but could not understand the problem, i thus took the initiative to build my own packages and update this at it was , well, old. I think deleting this directory could be the right thing to do. I, however, prefer the 'moved here' solution. |
Should read links in https://rpms.remirepo.net/temp/epel-temp.repo |
remicollet
left a comment
There was a problem hiding this comment.
Simple README.md LGTM.
|
Yup, i found it in the previous comment. Thanks for the head up. This does not sound good for the future of EPEL/PHP combo ... |
|
Merged, thanks! |
What:
This PR update the RPM packaging process. It uses now docker and the latest version of phpredis (5.3.0).
Why:
The RPM packaging was from 2014. It needed to be refreshed.