-
-
Notifications
You must be signed in to change notification settings - Fork 341
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
Could not install memcached-3.1.5 on PHP 7.4.15 #423
Comments
When I install |
Yes - name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: 7.4
extensions: memcached Test workflow: https://github.com/shivammathur/test-setup-php/runs/2021730562?check_suite_focus=true If you want to compile using pecl, you will have to install - name: Install required libraries
run: sudo apt-get install -y libmemcached-dev librdkafka-dev
- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: 7.4
tools: cs2pr
extensions: mbstring,bcmath,intl,gd,zip,memcached-3.1.5,redis-5.3.3,rdkafka-5.0.0 Test workflow: https://github.com/shivammathur/test-setup-php/actions/runs/617460339/workflow |
But it's weird that I had this config for a very long time without issues, and today it broke all of a sudden. When you specify the version, will it then always install from source? |
Since memcached is pre-installed and the input version matches like right now it both |
So this is the rule:
? |
Yes |
Then how come it broke today? Because I provide |
Can you please give me link to the logs where it is failing. |
I'm not able to reproduce the issue with PHP 7.4: https://github.com/shivammathur/test-setup-php/actions/runs/617493608/workflow |
Created a PR with my config, but it does not start the workflow: shivammathur/test-setup-php#3 |
The same happens with Redis - name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: '7.4'
tools: composer:v2, phpstan, redis
|
should be extension, no? |
@ruudk well it was working until today 🤷♂️ |
Ok, glad to hear I'm not crazy 😅 Something did change today |
I see some changes about Redis & other things between 2.9.0 and 2.10.0 What is strange is that Probably, this will work: uses: shivammathur/setup-php@2.9.0 |
@alexsegura See this comment shivammathur/test-setup-php#3 (comment) |
@alexsegura redis is not supported as a tool, it is supported as an extension. Please create a separate issue and provide your workflow. I will have a look. |
@shivammathur like I said, it used to work just like that. |
@ruudk if you install Both of these should work - name: Install required libraries
run: sudo apt-get install -y libmemcached-dev librdkafka-dev
- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: 7.4
extensions: memcached-3.1.5 - name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: 7.4
extensions: memcached |
Thanks for debugging this. I will add |
Yes, that can be improved in future release here. I will make a note of it. This issue was caused by change in how I will create a pull request to the actions images to get this fixed. |
Should be fixed, once this is merged and deployed |
@shivammathur Now that actions/runner-images#2834 is merged, should it work immediately? |
@ruudk No, it will have to be deployed first. There will be a release on actions/virtual-environment Ubuntu 20.04 20210304.2 or newer when they start deploying it. Till then please use #423 (comment) |
@ruudk Fix has been deployed and |
Describe the bug
memcached fails installing since today.
Version
2.10.0
Runners
Operating systems
Linux
PHP versions
7.4
To Reproduce
Expected behavior
This suddenly broke today.
Screenshots/Logs
The text was updated successfully, but these errors were encountered: