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.

Sign up for GitHub

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

Jump to bottom

Configuring keyboard creates user .config dir with owner root:root #267

Open
tboehler1 opened this issue Jan 14, 2025 · 1 comment · May be fixed by #268
Open

Configuring keyboard creates user .config dir with owner root:root #267

tboehler1 opened this issue Jan 14, 2025 · 1 comment · May be fixed by #268

Comments

@tboehler1
Copy link

When running raspi-config to configure the keyboard this creates the $HOMEDIR/.config/labwc folder (with mkdir -p) for the user running raspi-config. This folder is then chowned to $USER, as it should be.

If the $HOMEDIR/.config directory doesn't exist, though, this directory will still be owned by root:root afterwards.

If the $HOMEDIR/.config directory doesn't exist before doing this it should be created with the correct permissions in a separate step first.
tboehler1 added a commit to tboehler1/raspi-config that referenced this issue Jan 14, 2025
@tboehler1
fix: permission of created .config dir in labwc
a438684 
When the $HOMEDIR/.config/labwc folder is created it also creates the
$HOMEDIR/.config folder if it doesn't exist. As raspi-config is run with
sudo this makes the root user own the $HOMEDIR/.config directory.
Afterwards, only the $HOMEDIR/.config/labwc directory owner is changed
to $USER.

Add a function that ensures that the $HOMEDIR/.config directory exists
and, if it doesn't, create it with the correct permissions.

Fixes: RPi-Distro#267
Signed-off-by: Thomas Böhler <[email protected]>
@tboehler1 tboehler1 linked a pull request Jan 14, 2025 that will close this issue
Open
@AndrewFromMelbourne AndrewFromMelbourne mentioned this issue Jan 16, 2025
Closed
@AndrewFromMelbourne
Copy link

A raised an issue before a realized that this issue was created. My report has detailed instructions on how to reproduce the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: permission of created .config dir in labwc tboehler1/raspi-config
2 participants
@AndrewFromMelbourne @tboehler1