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

sudo: docker-php-ext-enable: command not found #166

Closed
Poonam0311 opened this issue Apr 13, 2022 · 8 comments · Fixed by #172
Closed

sudo: docker-php-ext-enable: command not found #166

Poonam0311 opened this issue Apr 13, 2022 · 8 comments · Fixed by #172
Assignees

Comments

@Poonam0311
Copy link

Poonam0311 commented Apr 13, 2022

On replacing the old image (circleci/php:7.4-node) with the new one(cimg/php:7.4-node), the circleci tests fail due to the below error:

sudo: docker-php-ext-enable: command not found

@JalexChen
Copy link
Contributor

Hi @Poonam0311 - these commands are not included on the new Cimgs and we are looking into adding this functionality.

@JalexChen JalexChen self-assigned this Apr 26, 2022
@acorncom
Copy link

👍 on adjusting things a tad. I was trying to use pcov with the 7.2 image as I upgraded a client's image and what formerly used to work:

          name: Enable PCOV
           command: |
             sudo pecl install pcov
             sudo -E docker-php-ext-enable pcov

no longer does. We lose all code coverage stats if we take that out

@joel-loycom
Copy link

Could we get an estimation on adding that? right now this is preventing us from using circle at the moment..

@JalexChen
Copy link
Contributor

Hi @acorncom and @joel-loycom - currently working on this and I'm hoping to get a PR out this week with the docker-php-ext-* commands available. Currently testing to ensure core and pecl extension installs are working correctly.

@acorncom
Copy link

@JalexChen thx much! happy to help test it against our use case if that'd be useful.

@joel-loycom
Copy link

That's awesome! thank you!

@JalexChen
Copy link
Contributor

@acorncom there is currently a PR up that you can check out: gh pr checkout 172. I specifically used your previous example, but it would be great for additional tests against your use case!

@JalexChen
Copy link
Contributor

Hi @acorncom @joel-loycom @Poonam0311

I've created three different versions of php for testing here.

If any of these versions/images are applicable to you, I would love if you could test them out for your use cases and report back on if any changes need to be made.

Appreciate it.

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 a pull request may close this issue.

4 participants