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

Updates to the APC Driver #256

Merged
merged 10 commits into from
Aug 9, 2015
Merged

Updates to the APC Driver #256

merged 10 commits into from
Aug 9, 2015

Conversation

tedivm
Copy link
Member

@tedivm tedivm commented Aug 9, 2015

This updates the APC driver, allowing the use (and testing against) of the APCu extension on modern versions of PHP.

tedivm added a commit that referenced this pull request Aug 9, 2015
@tedivm tedivm merged commit 5ae98e8 into v1.0.0-dev Aug 9, 2015
@tedivm tedivm deleted the apc_travis branch August 9, 2015 20:43
@layanto
Copy link

layanto commented Dec 6, 2015

This still rely on APC backward compatibility of APCu where all apc_* functions internally call apcu_* functions. APC backward compatibility is included by default for APCu on PHP5.5 and PHP5.6. But for PHP7, APC backward compatibility is not included by default (and still not working on Windows). Any chance to update the APC driver to check for existence of apcu_fetch function and if so use the apcu_* functions with fallback to apc_* functions?

@layanto
Copy link

layanto commented Dec 6, 2015

@tedivm
Copy link
Member Author

tedivm commented Dec 8, 2015

Can you open this as a new issue, rather than commenting here in this closed one?

@tedivm
Copy link
Member Author

tedivm commented Dec 8, 2015

Also, I'm currently waiting on Travis-CI to fix this PHP7 build. At the moment we can't test the apcu stuff on Travis due to them having disabled pecl on their php7 machines.

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.

2 participants