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

if [[ "$CURRENTUID" -ne "0" ]]; then HOME="/root" #393

Closed
gucio1200 opened this issue Jan 24, 2025 · 1 comment
Closed

if [[ "$CURRENTUID" -ne "0" ]]; then HOME="/root" #393

gucio1200 opened this issue Jan 24, 2025 · 1 comment

Comments

@gucio1200
Copy link

gucio1200 commented Jan 24, 2025

if [[ "$CURRENTUID" -ne "0" ]]; then
    HOME="/root"

This is definitely not correct , and just wont gonna work and not working.
Why setup HOME var for non root user to /root ?

@wolveix
Copy link
Owner

wolveix commented Jan 24, 2025

This is part of an ongoing exploration #389.

Per the comment immediately preceding the code you've highlighted:

# check if the user and group IDs have been set. If so, reset HOME to the upstream default


Essentially, we're trying to figure out a solution for the previous ROOTLESS implementation. The container requires root by default, and to supply your own --user parameter before you needed to set -e ROOTLESS=true. Of course, you're going to run into permissions issues (as the container rightfully warns you). I don't really see a way around this, outside of requiring people who wish to run it rootless to build it with new build arguments 🤷‍♂

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

No branches or pull requests

2 participants