Skip to content

Conversation

@jiangao1
Copy link

Solved the build problem in #31

@kantancoding
Copy link
Owner

Hi, can you write a description of what the issue was and exactly what your change does and how this solves the problem?

@jiangao1
Copy link
Author

jiangao1 commented Sep 27, 2024

Sure. When I run the docker build command with the existing Dockerfile in src/auth, the build fails with the error message: ERROR: failed to solve: process "/bin/sh -c pip install --no-cache-dir --requirement /app/requirements.txt"

Following the suggestions in the issue #31 by adding pkg-config, the docker build then becomes successful. docker version I have is 25.0.3

@kantancoding
Copy link
Owner

kantancoding commented Sep 27, 2024

Sure. When I run the docker build command with the existing Dockerfile in src/auth, the build fails with the error message: ERROR: failed to solve: process "/bin/sh -c pip install --no-cache-dir --requirement /app/requirements.txt"

Following the suggestions in the issue #31 by adding pkg-config, the docker build then becomes successful. docker version I have is 25.0.3

I see. Thank you! Can you tell me what pkg-config does?

@jiangao1
Copy link
Author

I'm not quite familiar with how it works, but from what I can gather, pkg-config is a tool that helps ensure compiler options are correct when compiling and linking software. It seems it helps to find where libraries are without knowing their exact path. https://www.freedesktop.org/wiki/Software/pkg-config/

@kantancoding
Copy link
Owner

I'm not quite familiar with how it works, but from what I can gather, pkg-config is a tool that helps ensure compiler options are correct when compiling and linking software. It seems it helps to find where libraries are without knowing their exact path. https://www.freedesktop.org/wiki/Software/pkg-config/

Got it! @jiangao1 Thank you!

@kantancoding kantancoding merged commit c3f4311 into kantancoding:main Oct 7, 2024
@jiangao1 jiangao1 deleted the fix-dockerfile branch October 7, 2024 18:07
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.

2 participants