ops-trust / portal

Ops-Trust Platform - Portal
https://www.ops-trust.net
Apache License 2.0
21 stars 4 forks source link

fix-install: Remove 'sudo' commands in fix-install #51

Closed teward closed 9 years ago

teward commented 9 years ago

This script should only be run as root, and should fail in all other cases.

In a stock Debian environment, the 'sudo' command does not exist, and the fix-install script will fail when attempting to call 'sudo'. Installation of 'sudo' will fix the problem, however if we are not intending to install 'sudo' and use a stock Debian type of setup, then this is a wrong approach. This also applies to other environments which may not even have the 'sudo' command available to it.

If you are running 'make install' which calls this 'fix-install' script, you're either already using 'sudo' to run this to install, or you are already 'root' and don't need to gain permissions. All other cases should fail since the command is not being run as superuser.