Skip to content

fix(build): pin to Debian bookworm-based images instead of trixie #0000 #20

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ayesh
Copy link
Member

@Ayesh Ayesh commented Aug 24, 2025

One of the packages these images install is the wkhtmltopdf, which is used by Onatal's legacy PDF renderer.

Since Debian Trixie was released on 2025 Aug, Docker Hub's PHP builds are based on Trixie. While this works for all other packages, this Debian version drops the then-obsolete wkhtmltopdf package:

To avoid build failures, this changes the PHP images to the ones tagged Debian bookworm.

Checklist

  • Breaking change: fix or feature that would cause existing functionality to not work as expected
  • I have read the Contributing doc
  • I have read the Creating and reviewing pull requests at LinkORB guide doc
  • Lint and unit tests pass locally with my changes
  • I have added/updated necessary documentation in the README.md or doc/ directories (if appropriate)

…0000

One of the packages these images install is the `wkhtmltopdf`, which is
used by Onatal's legacy PDF renderer.

Since Debian Trixie was released on 2025 Aug, Docker Hub's PHP builds
are based on Trixie. While this works for all other packages, this
Debian version drops the then-obsolete `wkhtmltopdf` package:

 - [`wkhtmltopdf` on `bookworm`](https://packages.debian.org/search?suite=bookworm&keywords=wkhtmltopdf)
 - [`wkhtmltopdf` on `trixie` (does not exist)](https://packages.debian.org/search?suite=trixie&keywords=wkhtmltopdf)

To avoid build failures, this changes the PHP images to the ones tagged
Debian bookworm.
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.

1 participant