Skip to content

Fix: Use uv pip install for Docling installation#12567

Merged
KevinHuSh merged 1 commit intoinfiniflow:mainfrom
LGRY:fix/docling-installation
Jan 13, 2026
Merged

Fix: Use uv pip install for Docling installation#12567
KevinHuSh merged 1 commit intoinfiniflow:mainfrom
LGRY:fix/docling-installation

Conversation

@LGRY
Copy link
Contributor

@LGRY LGRY commented Jan 12, 2026

Fixes #12440

What problem does this PR solve?

The current implementation uses python3 -m pip which can fail in certain environments. This change leverages uv pip install instead, which aligns with the project's existing tooling.

Type of change

  • Removed the ensurepip line (not needed since uv manages pip)
  • Changed python3 to "$PY" for consistency with the rest of the script
  • Changed python3 -m pip install to uv pip install

Fixes infiniflow#12440

The current implementation uses `python3 -m pip` which can fail in
certain environments. This change leverages `uv pip install` instead,
which aligns with the project's existing tooling.

Changes:
- Removed the ensurepip line (not needed since uv manages pip)
- Changed python3 to "$PY" for consistency with the rest of the script
- Changed python3 -m pip install to uv pip install
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. 🐞 bug Something isn't working, pull request that fix bug. labels Jan 12, 2026
@KevinHuSh KevinHuSh added the ci Continue Integration label Jan 13, 2026
@KevinHuSh KevinHuSh marked this pull request as draft January 13, 2026 02:43
@KevinHuSh KevinHuSh marked this pull request as ready for review January 13, 2026 02:43
@KevinHuSh KevinHuSh merged commit d76912a into infiniflow:main Jan 13, 2026
1 check passed
@LGRY LGRY deleted the fix/docling-installation branch January 16, 2026 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 bug Something isn't working, pull request that fix bug. ci Continue Integration size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Question]: using latest image(v0.23.1) can not enable docling

2 participants