forked from rvega/Fan-Control-Daemon
-
Notifications
You must be signed in to change notification settings - Fork 140
Closed
Description
I do not have sudo installed. Logging in as root I compiled and ran this tool, it suggested me to "sudo make test". Then I ran "make test", only to find it tries to sudo something. There sudo here is obviously redundant and should be removed (or at least test if the user actually has sudo).
I do understand that assuming the sudo command is available everywhere is probably reasonable. But I'd still like to point this out...
Metadata
Metadata
Assignees
Labels
No labels