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

fix(earthly): mv is buggy in debian | NPG-0000 #530

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

stevenj
Copy link
Collaborator

@stevenj stevenj commented Aug 23, 2023

mv will fail depending on the underlying filesystem of the volume used by docker for its volumes.

See: https://superuser.com/questions/1409618/renaming-a-file-with-mv-cannot-move-to-a-subdirectory-of-itself

This was failing for me, took forever to work it out what was actually happening.

That error report would indicate this will cause cross platform issues with the build. So lets not use mv in future.

…g on the file systems used by docker for its volumes.
@stevenj stevenj enabled auto-merge (squash) August 23, 2023 11:35
@stevenj stevenj requested a review from Mr-Leshiy August 23, 2023 11:35
@stevenj stevenj merged commit 6b47da6 into main Aug 23, 2023
27 checks passed
@stevenj stevenj deleted the bug/earthly-build-fixes branch August 23, 2023 12: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.

2 participants