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.

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

Fix memcache install for php 5.5 builds in travis #2124

Merged
merged 3 commits into from Feb 19, 2013
Merged

Fix memcache install for php 5.5 builds in travis #2124

merged 3 commits into from Feb 19, 2013

Conversation

arvenil
Copy link
Contributor

@arvenil arvenil commented Feb 18, 2013

Hi

As you can see here:
https://travis-ci.org/yiisoft/yii/jobs/4871285/#L179
Some test are skipped for php 5.5. Those are the one dependent on memcache.

In php 5.4. builds this is just ok:
https://travis-ci.org/yiisoft/yii/jobs/4871284/#L41

There was just small bug in installation of memcache:
https://github.com/arvenil/yii/commit/3c339515d5ccb565c9389e4802ffbe4716902d35#L0R9
(26786cc#L0R9)

I have also switched this script back to #!/bin/sh by removing one bashizm:
https://github.com/arvenil/yii/commit/367c6dce7a4367f6d13cc35095c31cf277334c9e#L0R3
(aea1241#L0L1)

And also silenced output for installing memcache in php 5.5 builds as this was done for previous versions:
https://github.com/arvenil/yii/commit/9a216ae3bb3a29ebd28ce8299897c72e74a75473#L0R17
(#931, #927)

In short: Tests in php 5.5 builds which use memcache should work now :)

mdomba added a commit that referenced this pull request Feb 19, 2013
Fix memcache install for php 5.5 builds in travis
@mdomba mdomba merged commit 94a5ee9 into yiisoft:master Feb 19, 2013
@mdomba
Copy link
Member

mdomba commented Feb 19, 2013

Merged.. thanks for working on this...

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.

None yet

2 participants