Skip to content

Alpine 3.13#173

Merged
tianon merged 1 commit intodocker-library:masterfrom
J0WI:alpine-3.13
Mar 23, 2021
Merged

Alpine 3.13#173
tianon merged 1 commit intodocker-library:masterfrom
J0WI:alpine-3.13

Conversation

@J0WI
Copy link
Copy Markdown
Contributor

@J0WI J0WI commented Jan 22, 2021

No description provided.

Comment thread 2.4/alpine/Dockerfile
runDeps=' \
apr \
apr-util \
apr-util-dbm_db \
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like this was added in #154 for mod_dav -- does that work without this now? 🙈 😇

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The package has been removed upstream and Alpine's apache2-webdav doesn't depend on it (only on apr-util). Might be still worth to double check. Do you know a common testcase?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So it likely will work fine since dbm_db (aka Berkeley DB <= 5.3) was replaced by ndbm in Alpine's build of apr:

-		--with-dbm=db53 \
-		--with-berkeley-db \
-		--without-gdbm
+		--without-berkeley-db \
+		--with-dbm=ndbm

LGTM

@tianon
Copy link
Copy Markdown
Member

tianon commented Mar 23, 2021

Doh, sorry; thank you! 👍

@tianon tianon merged commit 98fa1e7 into docker-library:master Mar 23, 2021
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Mar 24, 2021
Changes:

- docker-library/httpd@98fa1e7: Merge pull request docker-library/httpd#173 from J0WI/alpine-3.13
- docker-library/httpd@d36dfa5: Alpine 3.13
@J0WI J0WI deleted the alpine-3.13 branch March 24, 2021 17:51
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