Skip to content

add apr-util-ldap package to httpd:2.4-alpine (for LDAP authentication) #65

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 20, 2017
Merged

add apr-util-ldap package to httpd:2.4-alpine (for LDAP authentication) #65

merged 2 commits into from
Jul 20, 2017

Conversation

@Florent38
Copy link
Contributor

@Florent38 Florent38 commented Jul 19, 2017 •
edited by tianon
Loading

Closes #64

@Florent38 Florent38 mentioned this pull request Jul 19, 2017
Closed
@tianon
Copy link
Member

tianon commented Jul 19, 2017

Given that the size difference is essentially nothing, I'm +1 (~87.1MB vs ~87.1MB). 👍

@yosifkit ?

@tianon
Copy link
Member

tianon commented Jul 19, 2017

Oh, the size is no different because the package gets removed further down -- this doesn't actually include the package. 😅

@Florent38
Copy link
Contributor Author

Florent38 commented Jul 20, 2017

Sorry about that. New commit will fix this, I added the apr-util-ldap package in runDeps instead of buildDeps.

Size difference is minimal : 87104306 (with) vs 87093909 (without)

Package is correctly installed :

# apk info apr-util-ldap
apr-util-ldap-1.5.4-r3 description:
The Apache Portable Runtime Utility Library - LDAP driver

apr-util-ldap-1.5.4-r3 webpage:
http://apr.apache.org/

apr-util-ldap-1.5.4-r3 installed size:
28672

@yosifkit
Copy link
Member

yosifkit commented Jul 20, 2017

Given that this is basically #19. 👍

Should we also apply it to 2.2?

@yosifkit yosifkit merged commit c6ae30f into docker-library:master Jul 20, 2017
tianon added a commit to infosiftr/stackbrew that referenced this pull request Jul 21, 2017
@tianon
Update docker-library images
0b9f73e 
- `httpd`: add `apr-util-ldap` to Alpine for `mod_ldap` (docker-library/httpd#65)
- `memcached`: 1.5.0
- `openjdk`: windows ojdkbuild 1.8.0.141-1, add `slim` variants (docker-library/openjdk#133)
@tianon tianon mentioned this pull request Jul 21, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tianon tianon tianon approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Florent38 @tianon @yosifkit