Skip to content

Fix zwift install error with docker#341

Merged
netbrain merged 1 commit intonetbrain:masterfrom
glennvl:fix/docker-entrypoint
Apr 13, 2026
Merged

Fix zwift install error with docker#341
netbrain merged 1 commit intonetbrain:masterfrom
glennvl:fix/docker-entrypoint

Conversation

@glennvl
Copy link
Copy Markdown
Collaborator

@glennvl glennvl commented Apr 13, 2026

Summary

Checking the ownership of only the mount point itself appears to be not enough.

Fixes zwift install error when using docker:

wine: '/home/user/.wine' is not owned by you.

This is probably also the reason why the build from scratch workflow failed.

I remember that we discussed that this change was needed a while ago. And I thought it was actually already implemented. But I can't seem to find it in the git history, issues, discussions or pull requests.

Checking only the mount point itself appears to be not enough.
Fixes zwift install error with docker.
wine: '/home/user/.wine' is not owned by you.
@glennvl glennvl requested a review from netbrain April 13, 2026 16:25
@glennvl glennvl added the bug Something isn't working label Apr 13, 2026
@netbrain
Copy link
Copy Markdown
Owner

Hm weird..

@netbrain netbrain merged commit 2488207 into netbrain:master Apr 13, 2026
11 checks passed
@glennvl glennvl deleted the fix/docker-entrypoint branch April 13, 2026 17:16
@glennvl
Copy link
Copy Markdown
Collaborator Author

glennvl commented Apr 13, 2026

Hm weird..

Found where I read it 🙂.

@pdelagrave discovered this issue and mentioned it in #334.

entrypoint.sh

  • Fix ownership check depth (maxdepth 0maxdepth 1): mkdir -p $ZWIFT_HOME runs as root before privilege drop and creates .wine owned by root. The previous maxdepth 0 check on /home/user passed (it is owned by user) while missing the root-owned .wine subdirectory, causing Wine to refuse to start with "wine: '/home/user/.wine' is not owned by you".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants