Skip to content

fix(lychee,snipe-it): repair builds against Ubuntu 26.04 / PHP 8.5 base#2320

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/fix-ci-release-failure-lychee-snipe-it
Draft

fix(lychee,snipe-it): repair builds against Ubuntu 26.04 / PHP 8.5 base#2320
Copilot wants to merge 2 commits intomainfrom
copilot/fix-ci-release-failure-lychee-snipe-it

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 7, 2026

Two apps still failed to build after the Ubuntu 24.04 → 26.04 / PHP 8.5 migration in #2292 (run 25486727136). Both are fallout from the new base, not from upstream version changes.

Changes

  • apps/lychee/Dockerfile — Resolute ships ImageMagick 7, so the policy file lives at /etc/ImageMagick-7/policy.xml. The PDF coder sed injection was still pointing at the IM6 path and aborted the layer with sed: can't read /etc/ImageMagick-6/policy.xml: No such file or directory.

  • apps/snipe-it/Dockerfile — Upstream snipe-it v8.4.1's composer.lock caps several locked deps (lcobucci/jwt 5.5.0, nette/schema 1.3.2, phpspec/prophecy 1.22.0) at php ~8.4.0. Pass --ignore-platform-req=php to install the locked dep set unchanged on php8.5 — avoids composer update, which would silently drift the dep graph away from upstream's lockfile.

-  composer install --no-dev -d /app/www && \
+  composer install --no-dev --ignore-platform-req=php -d /app/www && \

Notes for review

  • No version bumps; both docker-bake.hcl VERSIONs untouched.
  • --ignore-platform-req=php is the conservative choice here: the upper bound in the lock is artificial (the libraries themselves work fine on 8.5), and snipe-it has no Resolute-compatible alternative since ppa:ondrej/php has no Resolute release (per fix(php-apps): align package names with Ubuntu 26.04 base #2292).
  • If/when upstream snipe-it ships a release whose lockfile permits 8.5, the flag can be dropped.

- lychee: Ubuntu 26.04 ships ImageMagick 7; policy.xml moved from
  /etc/ImageMagick-6/ to /etc/ImageMagick-7/. Update sed target path so
  the PDF coder policy injection succeeds.
- snipe-it: Upstream snipe-it v8.4.1 composer.lock pins several deps
  (lcobucci/jwt 5.5.0, nette/schema 1.3.2, phpspec/prophecy 1.22.0) with
  a php upper bound of ~8.4.0. Resolute's php8.5 trips the platform
  check. Pass --ignore-platform-req=php to install the locked dep set
  unchanged on PHP 8.5.

Both fixes are scoped to apps/<app>/ only. Validated with
`docker buildx bake --print` and full local image-local builds.

Refs https://github.com/trueforge-org/containerforge/actions/runs/25486727136

Agent-Logs-Url: https://github.com/trueforge-org/containerforge/sessions/cfb3fc2d-3a3d-4657-bf0c-71cb75042d92

Co-authored-by: Crow-Control <7613738+Crow-Control@users.noreply.github.com>
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.

fix(ci): release failure for lychee, snipe-it (run 25486727136)

2 participants