Fix Cargo Clean #398
Merged
Fix Cargo Clean #398
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Randy808
reviewed
Mar 27, 2024
|
I was still having some permission issues so I ended up executing a shell in the container as root with: and running: |
|
Is this a one time action or would you have to do it every time?
…On Wed, Mar 27, 2024, 7:38 PM Randy808 ***@***.***> wrote:
I was still having some permission issues so I ended up executing a shell
in the container as root with:
docker exec -u root -it <container_id> bash
and running:
chown -R <user> /tmp/gltesting
—
Reply to this email directly, view it on GitHub
<#398 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AATJLK6CTPE2UTEJRVBBCJTY2MGZFAVCNFSM6AAAAABFKW2ZQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRTGY3DANBVGQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
ErikDeSmedt
force-pushed
the
fix_cargo_clean
branch
from
d85c4dc
to
127f84f
Compare
|
It was a one time action |
Randy808
approved these changes
Mar 28, 2024
To reproduce the issue do `make docker-image`, `make docker-shell`, `cargo clean`. The issue is that you lack the rights to delete `tmp/gltesting/target`. The folder was added as a volume and cannot be deleted. We keep the rust target directory in `/tmp/gltesting/target/target` instead so it can be safely deleted.
ErikDeSmedt
force-pushed
the
fix_cargo_clean
branch
from
127f84f
to
ae3bdab
Compare
Randy808
approved these changes
Mar 28, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To reproduce the issue do
make docker-image,make docker-shell,cargo clean.The issue is that you lack the rights to delete
tmp/gltesting/target.The folder was added as a volume and cannot be deleted.
We keep the rust target directory in
/tmp/gltesting/target/targetinstead so it can be safely deleted.
I'd recommend to run
rm -rf /tmp/gltestingon the host-machine after merging this update.