Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change WORKDIR from / to /usr/src/app #141

Merged
merged 3 commits into from
Aug 3, 2023

Conversation

waterkip
Copy link
Contributor

@waterkip waterkip commented Aug 2, 2023

What changed is that in bookworm the lib dir has become a symlink to /usr/lib.
This prevents COPY . . to copy over our lib dir to the layer.

For more information:

Closes: #140

Signed-off-by: Wesley Schwengle waterkip@cpan.org

What changed is that in bookworm the lib dir has become a symlink to /usr/lib.
This prevents `COPY . .` to copy over our lib dir to the layer.

For more information:
* https://wiki.debian.org/NewInBookworm
* https://wiki.debian.org/UsrMerge
* https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=978636:

Closes: Perl#140

Signed-off-by: Wesley Schwengle <waterkip@cpan.org>
Signed-off-by: Wesley Schwengle <waterkip@cpan.org>
@zakame
Copy link
Member

zakame commented Aug 2, 2023

LGTM, thanks @waterkip ! 🙏

Can we add a test in https://github.com/Perl/docker-perl/blob/master/.github/workflows/build-image.yml based on what we've observed (so on the off chance we revert WORKDIR or some other change happens, we can can catch this issue later?)

Signed-off-by: Wesley Schwengle <waterkip@cpan.org>
@waterkip
Copy link
Contributor Author

waterkip commented Aug 2, 2023

Added the test.

It might be considered a breaking change. For example, if someone did COPY foo . and later on does RUN /foo it will stop working.

Copy link
Member

@zakame zakame left a comment

Choose a reason for hiding this comment

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

LGTM! 👍 thanks for this - as for the breaking change, I think Debian broke us first 😹 we'll just make an update on our doc and move forward 🚀

@zakame zakame merged commit e609668 into Perl:master Aug 3, 2023
42 checks passed
@waterkip
Copy link
Contributor Author

waterkip commented Aug 3, 2023

LGTM! +1 thanks for this - as for the breaking change, I think Debian broke us first joy_cat we'll just make an update on our doc and move forward rocket

Haha, ok ok :)

@waterkip waterkip deleted the GL-140-docker_workdir branch August 3, 2023 15:13
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.

Simple Dockerfile does not work anymore
2 participants