[PR #5293] [MERGED] simpifly Dockerfiles #6463

Closed
opened 2026-02-05 10:32:58 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/BookStackApp/BookStack/pull/5293
Author: @johnroyer
Created: 10/28/2024
Status: Merged
Merged: 12/1/2024
Merged by: @ssddanbrown

Base: developmentHead: docker-simplify


📝 Commits (2)

📊 Changes

1 file changed (+27 additions, -26 deletions)

View changed files

📝 dev/docker/Dockerfile (+27 -26)

📄 Description

some updates:

  • Remove deprecated syntax
  • use mlocati/docker-php-extension-installer to install PHP extensions: easier to read and install dependencies automatically
  • use multiple docker images: docker image layers are re-usable, it can accelerate build speed while working on Dockerfile

Notes: everything should be works on docker compose up


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/BookStackApp/BookStack/pull/5293 **Author:** [@johnroyer](https://github.com/johnroyer) **Created:** 10/28/2024 **Status:** ✅ Merged **Merged:** 12/1/2024 **Merged by:** [@ssddanbrown](https://github.com/ssddanbrown) **Base:** `development` ← **Head:** `docker-simplify` --- ### 📝 Commits (2) - [`4b60c03`](https://github.com/BookStackApp/BookStack/commit/4b60c03caa5ff990bf935b4b6ede7d8d32f0e8c5) re-write Dockerfile - [`fbeb2e2`](https://github.com/BookStackApp/BookStack/commit/fbeb2e23d41b381d0cf15ab0e06f8109497fb2a1) fix deprecated syntax ### 📊 Changes **1 file changed** (+27 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `dev/docker/Dockerfile` (+27 -26) </details> ### 📄 Description some updates: - Remove deprecated syntax - use `mlocati/docker-php-extension-installer` to install PHP extensions: easier to read and install dependencies automatically - use multiple docker images: docker image layers are re-usable, it can accelerate build speed while working on Dockerfile Notes: everything should be works on `docker compose up` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 10:32:58 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#6463