Skip to content

Fix pdf2any Docker workflow dependency resolution - #1

Merged
ayang merged 2 commits into
masterfrom
copilot/fix-pdf2any-docker-job
Jul 2, 2026
Merged

Fix pdf2any Docker workflow dependency resolution#1
ayang merged 2 commits into
masterfrom
copilot/fix-pdf2any-docker-job

Conversation

Copilot AI commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

The pdf2any-docker Actions job was failing before test execution because the container install step still referenced libgl1-mesa-glx, which is no longer available in the current python:3.10 image. Once that was unblocked, the job also surfaced a missing declared dependency for PIL.

  • CI package update

    • Replace libgl1-mesa-glx with libgl1 in the workflow system dependency install steps.
    • Keep the Docker and Ubuntu jobs aligned on the same OpenGL runtime package.
  • Runtime dependency declaration

    • Add Pillow to requirements.txt to match the existing from PIL import Image as PILImage import path used by the package.
  • Net effect

    • The Docker job can progress past OS package installation and resolve the image-handling dependency required by the conversion code path.
- run: apt-get update && apt-get install -y libgl1
opencv-python>=4.5
Pillow>=12.3.0
fire>=0.3.0

Copilot AI changed the title [WIP] Fix failing GitHub Actions job pdf2any-docker Fix pdf2any Docker workflow dependency resolution Jul 2, 2026
Copilot AI requested a review from ayang July 2, 2026 15:33
Copilot finished work on behalf of ayang July 2, 2026 15:33
@ayang
ayang requested review from ayang and Copilot July 2, 2026 15:40
@ayang ayang closed this Jul 2, 2026
@ayang ayang reopened this Jul 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes CI failures in the pdf2any-docker GitHub Actions job by updating the OS package installed in the container and by declaring a missing runtime dependency needed for image processing paths in pdf2any.

Changes:

  • Update CI system dependency installation to use libgl1 instead of libgl1-mesa-glx (Docker + Ubuntu jobs).
  • Add Pillow to requirements.txt to satisfy the existing from PIL import Image as PILImage import used by the image conversion code.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
requirements.txt Declares Pillow as a runtime dependency to support existing PIL-based image handling.
.github/workflows/test.yml Updates apt package installation to libgl1 so the Docker job can install OpenGL runtime deps successfully.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ayang
ayang marked this pull request as ready for review July 2, 2026 15:42
@ayang
ayang merged commit fe04fb3 into master Jul 2, 2026
1 check passed
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