Skip to content

"Please run as root" msg during install #333

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

Closed
LightningStalker opened this issue Jun 8, 2024 · 1 comment · Fixed by #334
Closed

"Please run as root" msg during install #333

LightningStalker opened this issue Jun 8, 2024 · 1 comment · Fixed by #334
Labels
conclusion: resolved Issue was resolved topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project

Comments

@LightningStalker
Copy link

AppImage version
No indications but appears to be the post_install.sh
Running IDE suid or sudo -s homes as root instead of user home
sudo -s echo $HOME = /home/user

@per1234 per1234 added type: imperfection Perceived defect in any part of project topic: infrastructure Related to project infrastructure labels Jun 9, 2024
@per1234
Copy link
Collaborator

per1234 commented Jun 9, 2024

Hi @LightningStalker.

appears to be the post_install.sh

That is correct:

then echo "Please run as root"

Running IDE suid or sudo -s homes as root instead of user home

Yeah, you should not run Arduino IDE as superuser. I think this message was intended for those who are running the script directly from the terminal. However, when it is shown by Arduino IDE the message is misleading and unhelpful.

I submitted a proposal to make the output more friendly: #334

@per1234 per1234 added the conclusion: resolved Issue was resolved label Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: resolved Issue was resolved topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants