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

Clean up a few extraneous bits leftover from our php:apache transition #25

Merged
merged 1 commit into from
Nov 11, 2014
Merged

Clean up a few extraneous bits leftover from our php:apache transition #25

merged 1 commit into from
Nov 11, 2014

Conversation

tianon
Copy link
Member

@tianon tianon commented Nov 11, 2014

No description provided.

@yosifkit
Copy link
Member

LGTM

yosifkit added a commit that referenced this pull request Nov 11, 2014
Clean up a few extraneous bits leftover from our php:apache transition
@yosifkit yosifkit merged commit 593c91c into docker-library:master Nov 11, 2014
@yosifkit yosifkit deleted the cleanup-wordpress branch November 11, 2014 18:04
@tianon
Copy link
Member Author

tianon commented Nov 11, 2014

For the casual observers: this is so that we push most of the logic for getting PHP+HTTP working off to php:apache, where it belongs.

tianon added a commit to infosiftr/stackbrew that referenced this pull request Nov 12, 2014
- `php`: fix some Apache woes (docker-library/php#43), and add `docker-php-ext-install` and `docker-php-ext-configure` (docker-library/php#41)
- `tomcat`: 7.0.57
- `wordpress`: push more of the PHP+Apache logic into `php` where it belongs (docker-library/wordpress#25), and update the image to use the new `docker-php-ext-install` magic for the extra core extensions it needs (docker-library/wordpress#26)
tianon added a commit to infosiftr/stackbrew that referenced this pull request Nov 12, 2014
- `php`: fix some Apache woes (docker-library/php#43), and add `docker-php-ext-install` and `docker-php-ext-configure` (docker-library/php#41)
- `tomcat`: 7.0.57
- `wordpress`: push more of the PHP+Apache logic into `php` where it belongs (docker-library/wordpress#25), and update the image to use the new `docker-php-ext-install` magic for the extra core extensions it needs (docker-library/wordpress#26)
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