Skip to content

Add alpine fpm variant #72

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

Merged
merged 3 commits into from
Mar 1, 2017
Merged

Add alpine fpm variant #72

merged 3 commits into from
Mar 1, 2017

Conversation

sylus
Copy link
Contributor

@sylus sylus commented Feb 6, 2017

Just adding some awesome alpine support. ^_^

I didn't touch the apache conf given:

docker-library/php#259

I also updated the update.sh script but wasn't sure if any changes needed to the generate-stackbrew-library.sh file.

Closes #73

@sylus sylus mentioned this pull request Feb 6, 2017
@sylus sylus changed the title Add alpine fpm variants Add alpine fpm variant Feb 6, 2017
@tianon
Copy link
Member

tianon commented Mar 1, 2017

Hey @sylus, thanks for the contribution and your patience ❤️

I was reviewing this and noticed a bunch of consistency changes I wanted to make around this, so I took the liberty of just doing those on top of your additions (so that your additions could be included too). 👍 ❤️

Hopefully we can get this in shortly! (I've asked @yosifkit to take a look now)

@yosifkit yosifkit merged commit 74de2d0 into docker-library:master Mar 1, 2017
tianon added a commit to infosiftr/stackbrew that referenced this pull request Mar 1, 2017
- `drupal`: 8.3.0-rc1, add `fpm-alpine` variants (docker-library/drupal#72)
- `elasticsearch`: 5.2.2
- `ghost`: 0.11.7
- `golang`: remove 1.6
- `haproxy`: 1.7.3
- `kibana`: 5.2.2
- `logstash`: 5.2.2
- `piwik`: 3.0.2
@sylus
Copy link
Contributor Author

sylus commented Mar 1, 2017

Thank you so much @tianon and @yosifkit! Really appreciate all the awesome work you do!

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.

3 participants