Skip to content
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.

Sign up for GitHub

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

Jump to bottom

php71u-pecl-memcached #111

Closed
vasillalov opened this issue Dec 23, 2016 · 27 comments
Closed

php71u-pecl-memcached #111

vasillalov opened this issue Dec 23, 2016 · 27 comments
Labels
php71u

Comments

@vasillalov
Copy link

What package do you want?

php71u-pecl-memcached

Why?

Provide ability to store sessions and binary data in memcached clusters. Memcached scales better than Redis and has better mutex capabilities.

A critical dependency (php71u-pecl-igbinary) has been provided by the upstream vendor. At present, there are debian packages, but none exist for RHEL/CentOS distros.

OS versions

I would like this package for both RHEL 7, CentOS 7

Testing

I agree to test the new package to ensure that it works as expected.
@carlwgeorge
Copy link
Member

Hello @vasillalov,

Unfortunately this request is affected by the same problem detailed in the request for php70u-pecl-memcached. There is no stable upstream release that supports PHP7 yet. There isn't even a beta release for us to work with. Checking open issues, it seems that their php7 development branch still has segfaults. We're going to have to put this request on hold.

@jpackagejasonc
Copy link

looks like there should be a release today (or if not today, very very soon): https://github.com/php-memcached-dev/php-memcached/blob/master/ChangeLog#L3

@carlwgeorge
Copy link
Member

happening

@vasillalov
Copy link
Author

vasillalov commented Jan 27, 2017 via email

@badrange badrange mentioned this issue Jan 30, 2017
Closed
@carlwgeorge
Copy link
Member

3.0.1 is on PECL now, so we will proceed with packaging this.

@carlwgeorge
Copy link
Member

@vasillalov, your original request was only for EL7. Is there any interest from you (or anyone else for that matter) to have this for EL6 as well?

@jpackagejasonc
Copy link

@carlwgeorge definite interest from me for EL6, thanks!

@vasillalov
Copy link
Author

@carlwgeorge support for EL7 is primary focus for our organization. However, we have several customers who are still on EL6 and having support for EL6 would be of great help. This will allow upgrades of the LAMP stack without the need to upgrade the OS which could be difficult and lengthy process.

We will test EL7 packages first and EL6 second.

Thank you for your support and help.

@carlwgeorge
Copy link
Member

Here is our spec file. For the initial build, we are only going to target EL7. Our decision around EL6 packages is contingent on how we handle libmemcached10 going forward. The initial EL7 packages will move to the testing repositories during tonight's automation.

@jpackagejasonc
Copy link

@carlwgeorge I can definitely see why you made that decision regarding EL6 now that I've built the packages on my own from the EL7 spec. I still would like to see it added though, so just restating my request for EL6 support eventually. thanks!

@carlwgeorge
Copy link
Member

@jpackagejasonc Don't lose hope, we haven't made a final decision yet. I just didn't want to delay the EL7 packages due to the situation around libmemcached10 on EL6. We want to avoid over-extending ourselves by keeping packages around forever, especially considering that EL6 will start Production 3 Phase on 2017-05-10 (in about three months). IUS has a general rule to no longer accept new packages requests for an EL version once it reaches Production 3 Phase, so that weighs into our decision as well since EL6 is so close to that phase.

@jpackagejasonc
Copy link

@carlwgeorge makes sense to me. if you do add el6 support will you update this bug or do I need to keep an eye on the repo? and would a PR to add it help at all? I've already done the work and could clean it up for your review

@carlwgeorge
Copy link
Member

I'll update this issue again.

Thanks for the PR offer, but it isn't necessary. I already have an uncommitted spec file with the appropriate changes.

@jpackagejasonc
Copy link

awesome, thanks!

@carlwgeorge carlwgeorge mentioned this issue Feb 13, 2017
Closed
@carlwgeorge
Copy link
Member

I have updated the spec file to 3.0.3 to address this issue. New packages will move to the testing repositories during tonight's automation.

@jpackagejasonc
Copy link

@carlwgeorge still el7 only for now?

@carlwgeorge
Copy link
Member

@jpackagejasonc Yes.

@ammeil
Copy link

ammeil commented Feb 22, 2017

Is there still hope to get this for EL6 ?

@tvlooy
Copy link

tvlooy commented Mar 1, 2017

@carlwgeorge what is holding this back to be committed for EL7? I'm not familiar with iuscommunity way of working

@carlwgeorge
Copy link
Member

@tvlooy Before promoting new packages to the stable repos, we require user feedback that the packages work as expected (usually from the original requester, but it can be from anyone willing to help test). Have you tried the packages? Do they work for you? We don't need confirmation that every possible thing works, just a basic "works for me" statement (similar to the karma system for Fedora and EPEL packages).

@ammeil
Copy link

ammeil commented Mar 2, 2017

I have tested this package and it appears to be working fine. Thanks @carlwgeorge

@carlwgeorge
Copy link
Member

Thanks for the feedback @ammeil. The php71u-pecl-memcached-3.0.3-1 EL7 packages will be promoted to our stable repositories during tonight's automation.

@carlwgeorge
Copy link
Member

I realize that many users would still like to get this package built for EL6. We are still evaluating if we are going to proceed with that, but if we do this issue will be updated again (despite the closed status).

@cmkl
Copy link

cmkl commented Jan 14, 2019

I could really use this for CentOS 6. I've moved to PHP 7.1 using IUS repository's php71u and there's no php-memcached so all my hosted applications (Wordpress, Drupal, Joomla) have lost the ability to use memecached.

@carlwgeorge carlwgeorge mentioned this issue Jan 15, 2019
Closed
@carlwgeorge
Copy link
Member

carlwgeorge commented Jan 15, 2019 •
edited

Sorry we were never able to complete this for EL6.

To recap the situation, pecl-memcached needs libmemcached, but the version in EL6 is too old. In the past we solved for this with libmemcached10. IUS would like to deprecate that package because it's difficult to maintain. It's stuck at 1.0.16 because 1.0.18 (the latest) is only 59% backwards compatible on an API/ABI level. As it stands, the only thing requiring libmemcached10 is php56u-pecl-memcached, which will be retired at the end of the month.

If someone adds a parallel-installable libmemcached 1.0.18 to EPEL for EL6 (using a name like libmemcached1), I can try to build against that. I just don't have the bandwidth to take that on myself.

@cmkl
Copy link

cmkl commented Jan 15, 2019 via email

@carlwgeorge
Copy link
Member

Better late than never, php71u-pecl-memcached is now available for EL6 in the testing repo on the new IUS CDN.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
php71u
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@tvlooy @jpackagejasonc @cmkl @ammeil @carlwgeorge @vasillalov