Skip to content

Update docker.md#810

Open
purpose-pending wants to merge 1 commit into
matter-js:mainfrom
purpose-pending:patch-1
Open

Update docker.md#810
purpose-pending wants to merge 1 commit into
matter-js:mainfrom
purpose-pending:patch-1

Conversation

@purpose-pending

Copy link
Copy Markdown

chown and chmod flags have to be uppercase

chown and chmod flags have to be uppercase

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Docker troubleshooting documentation to use the correct (uppercase) recursive flags for ownership and permission commands, so users can reliably fix volume permission issues when running the server in a container.

Changes:

  • Replace invalid -r flags with the correct recursive -R for chown and chmod.
  • Ensure the documented commands actually apply to the full data/ directory tree.

Comment thread docs/docker.md
chown -r 1000:1000 data
chmod -r 755 data
chown -R 1000:1000 data
chmod -R 755 data

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

hm ... yeahh x is usually not needed ...

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.

3 participants