Skip to content

Fix build error by copying src before install#16

Merged
marevol merged 2 commits intocodelibs:mainfrom
ytachiki:fix/dockerfile-copy-src
Aug 14, 2025
Merged

Fix build error by copying src before install#16
marevol merged 2 commits intocodelibs:mainfrom
ytachiki:fix/dockerfile-copy-src

Conversation

@ytachiki
Copy link
Contributor

No description provided.

@ytachiki ytachiki marked this pull request as draft August 13, 2025 09:48
@marevol marevol requested review from Copilot and marevol and removed request for Copilot August 14, 2025 01:04
Copy link
Contributor

Copilot AI left a comment

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 a Docker build error by reordering the COPY operations to ensure the source code is available before attempting to install the package. The key change moves the COPY src/ src/ command before the RUN uv pip install command, which requires the source files to be present for a successful installation.

  • Moved source code copy operation before package installation
  • Removed redundant copy operation from the final image stage
Comments suppressed due to low confidence (1)

Dockerfile:29

  • Removing this COPY command may break the application if the installed package in /build/python doesn't include the fessctl module or if it's installed in a different location. Verify that the fessctl module is properly accessible in the final image after installation.
COPY --from=builder /build/python /app/

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@marevol marevol marked this pull request as ready for review August 14, 2025 01:05
@marevol marevol merged commit bafca5e into codelibs:main Aug 14, 2025
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