Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

Error on install as root #1178

Merged
merged 1 commit into from
Jul 27, 2021
Merged

Error on install as root #1178

merged 1 commit into from
Jul 27, 2021

Conversation

davidvonthenen
Copy link
Contributor

What this PR does / why we need it

Do not allow the use to run the Linux install.sh script as root.

Which issue(s) this PR fixes

#1171

Describe testing done for PR

Tested ./install.sh works
Tested sudo ./install.sh errors out as expected

Special notes for your reviewer

No

Does this PR introduce a user-facing change?

No

@karuppiah7890 karuppiah7890 mentioned this pull request Jul 27, 2021
Closed
stmcginnis
stmcginnis approved these changes Jul 27, 2021
View reviewed changes
Copy link
Contributor

@stmcginnis stmcginnis left a comment

Choose a reason for hiding this comment

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

We may want to be more verbose with the error output to give a little more guidance, but I think this is good enough as it is.

@davidvonthenen davidvonthenen merged commit ff6da79 into main Jul 27, 2021
@davidvonthenen davidvonthenen deleted the feature/noroot branch July 27, 2021 15:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@stmcginnis stmcginnis stmcginnis approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@davidvonthenen @stmcginnis