search

ergochat / ergo

A modern IRC server (daemon/ircd) written in Go.
https://ergo.chat/
MIT License
2.21k stars 176 forks source link

Update `MANUAL.md` for the Debian 12 syntax. #2080

Closed KoxSosen closed 10 months ago

KoxSosen commented 1 year ago

On Debian 12 adduser doesn't create a home directory for a system user by default, instead the "home" directory will be /nonexistent. To create a home directory, we need to explicitly specify the --home parameter and point it to a directory.

Additionally, chown fails with the syntax from the manual. Instead of using a wildcard, specifying the -R parameter should recursively change ownership of directories, which seems perfect in this case.

I'm not sure if it's the correct decision to update the manual because of backwards compatibility issues, but I figured to open a pull request still. If nothing more, a note would be great in my opinion.

slingamn commented 1 year ago

Ah yes, the shell glob is evaluated with non-root permissions.

slingamn commented 10 months ago

I missed that this PR was opened against stable; in future, please open PRs against master unless there's a clear reason to use another branch.

KoxSosen commented 10 months ago

Noted, and thank you merging!