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

Update Dockerfile because php:apache updated #86

Merged
merged 2 commits into from Jul 3, 2018
Merged

Conversation

zuphilip
Copy link
Member

@zuphilip zuphilip commented Jan 5, 2018

The php:apache now uses also Debian 7 with PHP 7 and therfore
we need to replace the package php5-dev with php-dev.

See also #85.

The php:apache now uses also Debian 7 with PHP 7 and therfore
we need to replace the package php5-dev with php-dev.

See also #85.
@zuphilip
Copy link
Member Author

zuphilip commented Jan 5, 2018

Hm.. not yet working, see https://hub.docker.com/r/ubma/malibu/builds/bwl5j9ddkttfwwqhczwm2nu/
"
Package 'php-dev' has no installation candidate E: Package 'php-pear' has no installation candidate �[0m Package php-pear is not available, but is referred to by another package. This may mean that the package is missing, has been obsoleted, or is only available from another source Package php-dev is not available, but is referred to by another package. This may mean that the package is missing, has been obsoleted, or is only available from another source
"

@zuphilip
Copy link
Member Author

zuphilip commented Jan 6, 2018

Okay, the docker image build successfully now without the packages php-dev and php-pear: https://hub.docker.com/r/ubma/malibu/builds/. I have read that

When you install php* packages from Debian, you install a second copy of PHP (or at least some of its modules), and not the version you're intending to use from this image.

Are these packages not anymore needed at all in Debian? CC @stweil

@zuphilip
Copy link
Member Author

@stweil Can you look at this PR and also possible changes for our installation on a Debian system? For more information see https://github.com/docker-library/docs/tree/master/php#e-package-php-xxx-has-no-installation-candidate

@zuphilip zuphilip merged commit 0f10d7d into master Jul 3, 2018
@zuphilip zuphilip deleted the update-dockerfile branch July 3, 2018 16:09
@zuphilip
Copy link
Member Author

zuphilip commented Jul 3, 2018

Merged 🎉 CC @jmechnich

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

1 participant