Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Upstream 2.2.0 version #1

Closed
Xon opened this issue Jul 4, 2014 · 12 comments
Closed

Upstream 2.2.0 version #1

Xon opened this issue Jul 4, 2014 · 12 comments

Comments

@Xon
Copy link

Xon commented Jul 4, 2014

memcached has a 2.2.0 stable (http://pecl.php.net/package/memcached). Would it be possible to get this pushed into IUS repo for it's php 5.5 build?

@b-harper
Copy link
Contributor

b-harper commented Aug 5, 2014

Hello Xon,

Sorry for the delay. Somehow, we did not get notified when you created this issue.

We received a request for this package late last year. With new package requests, we require that the requester (or someone else) to test the RPM before we place it into the stable repos. We have not gotten any feedback and this package got moved to the development repos.

If we were to updated this package to 2.2.0 stable could you test it and give feedback?

We are tracking this package request in the following launchpad bug:

https://bugs.launchpad.net/ius/+bug/1258608

@Xon
Copy link
Author

Xon commented Aug 6, 2014

Hi,

I wouldn't have a problem with that.

My plan is to test it on 1 or 2 servers out of 3 before rolling it out to all the nodes so this would work for me.

Regards,
Xon

@b-harper
Copy link
Contributor

b-harper commented Aug 6, 2014

Hey Xon,

If that is the case, I'll work on getting some updated packages.

@Xon
Copy link
Author

Xon commented Sep 10, 2014

Hi,

Has there been any progress? Or would be waiting for a php 5.6 version be ideal?

I can only see 2.1.0 in the dev repo.

Regards,
Xon

@cgtx
Copy link
Contributor

cgtx commented Sep 10, 2014

I updated the spec file in this commit: 3c34f8a.

However, we are waiting to see what the resolution to this upstream issue will be: php-memcached-dev/php-memcached#146

@cgtx
Copy link
Contributor

cgtx commented Sep 10, 2014

@cgtx
Copy link
Contributor

cgtx commented Sep 10, 2014

We found out that upstream is building and testing against the following versions of libmemcached:

  • 1.0.17, 1.0.16, 1.0.15, 1.0.14, 1.0.10, 1.0.8, 1.0.7, 1.0.6, 1.0.2
  • 0.53, 0.49, 0.44

Noticable absent is 0.31, which is what is stock on EL 6. We do have the IUS package libmemcached10, so I created a build against that with the latest php module source. It will be available in the development repo tonight. We would prefer to switch the package back to using stock libmemcached, but we will have to see what upstream does first.

@Xon
Copy link
Author

Xon commented Sep 11, 2014

Hi,

Thanks for the heads up on the issues.

@Xon
Copy link
Author

Xon commented Oct 14, 2014

@b-harper I'll organise testing against this ASAP, missed that it had migrated to the testing repo

@Xon
Copy link
Author

Xon commented Oct 14, 2014

PHP Warning: PHP Startup: Unable to load dynamic library '/usr/lib64/php/modules/memcached.so' - /usr/lib64/php/modules/memcached.so: undefined symbol: php_json_decode_ex in Unknown on line 0

Copying /etc/php.d/json.ini to /etc/php.d/40.json.ini fixed the dependency issue

@cgtx
Copy link
Contributor

cgtx commented Oct 14, 2014

Howdy Xon,

Sorry about that; let me clarify on the issue you saw. Another user request that we enable igbinary support. This change requires that modules be loaded in a certain order, which is why Fedora migrated to a numbered scheme for their ini files. However, just enumerating memcached (50-memcached.ini) and igbinary (40-igbinary.ini) was not enough, since we need json to load before memcached as well. It worked before just based on alphabetical order (j comes before m), but I broke it when I renamed the memcached ini file (4 comes before j).

The current testing build of php55u-pecl-jsonc renames the ini file to 40-json.ini. In standard RPM fashion, the change will happen automatically if the existing json.ini file is unchanged, and if it has been modified it will be renamed as json.ini.rpmsave, and the new 40-json.ini will be created.

After making these changes, I decided the correct thing to do is to fully convert our php55u series to the enumerated ini scheme. Every php55u package has been rebuilt with this change, and is available in the testing repos. If you would like to help, please update these packages and provide us feedback that they work as expected.

yum --enablerepo=ius-testing update php55u*

I contacted the upstream libmemcached developers about the original issue of building against version 0.31. They have no interest in making it work, so we will continue to build against the IUS libmemcached10 package.

@Xon
Copy link
Author

Xon commented Oct 15, 2014

Hi cgtx,

Glad hear about igbinary support is in.

I've updated the packages and it's all working well. I'm hoping to get the memcached extension used soon.

php55u-pecl-memcache is working just fine.

Haven't finished testing php55u-pecl-xdebug yet.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants