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

docs: Fix docker run command to fix Uncaught PermissionError #2902

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mikegehard
Copy link

Fixes #2891
Fixes #2865

@paul-gauthier
Copy link
Collaborator

Users have reported needing those switches to ensure sane file ownership/perms in the host os?

@mikegehard
Copy link
Author

mikegehard commented Jan 21, 2025

@paul-gauthier Yeah not sure why the user id and group id on your local machine would affect the user id and group id in the running container. 🤷

I have a feeling that there are some weirdness in the current docker container with file permissions in general because stuff gets installed outside of the home directory.

I did some refactoring of the Dockerfile to simplify it here and hopefully clean up some of the permissions: #2905

Specifically I do a chmod at the end to change ownership to the container user.

I'm happy to do more Aider development in that container using the devcontainer to make sure there aren't any issues with permissions before you merge it all into main? If you could also do the same, I bet it could be stress tested enough to say that using the Docker image in the above commit is working as expected.

Let me know how I can help. What you've created here is amazing and I want to contribute back.

I've started to move all of my development using Aider into devcontainers but since I don't do a bunch of Python work, I'm using Docker containers where I'm installing Aider with uv.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants