From b26a87a33c4487c98f7cb8e6a5279feb4c79eb16 Mon Sep 17 00:00:00 2001 From: Peter Eckersley Date: Wed, 11 Nov 2015 12:57:32 -0800 Subject: [PATCH] Comments on SUDO --- letsencrypt-auto | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/letsencrypt-auto b/letsencrypt-auto index 667dcecfd..160de036a 100755 --- a/letsencrypt-auto +++ b/letsencrypt-auto @@ -27,11 +27,16 @@ for arg in "$@" ; do fi done +# letsencrypt-auto needs root access to bootstrap OS dependencies, and +# letsencrypt itself needs root access for almost all modes of operation +# The "normal" case is that sudo is used for the steps that need root, but +# this script *can* be run as root (not recommended), or fall back to using +# `su` if test "`id -u`" -ne "0" ; then if command -v sudo 1>/dev/null 2>&1; then SUDO=sudo else - # `sudo` command does not exist, use `su` instead. + echo \"sudo\" is not available, will use \"su\" for installation steps... # Because the parameters in `su -c` has to be a string, # we need properly escape it su_sudo() {