From aa2b21a5a69d2458fb1980d3397f54dcdc4c7368 Mon Sep 17 00:00:00 2001 From: David Goodwin Date: Sun, 24 Oct 2021 11:24:45 +0100 Subject: [PATCH] try and improve install.sh - PATH, check for php, better error/warning message etc --- install.sh | 21 +++++++++++++++++++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/install.sh b/install.sh index 4785fb32..116e2cba 100644 --- a/install.sh +++ b/install.sh @@ -2,9 +2,18 @@ set -eu +# PostfixAdmin install script. +# 1. Downloads 'composer.phar' to the current directory. +# 2. Runs 'php composer.phar install' which should install required runtime libraries for Postfixadmin +# 3. Runs 'mkdir templates_c && chmod 777 templates_c' + +PATH=/bin:/usr/bin:/usr/local/bin +export PATH COMPOSER_URL=https://getcomposer.org/download/latest-stable/composer.phar +type php >/dev/null 2>&1 || { echo >&2 "I require php but it's not installed. Aborting."; exit 1; } + cd "$(dirname "$0")" # Check for $(pwd)/composer.phar @@ -34,10 +43,18 @@ php composer.phar install --prefer-dist -n --no-dev if [ ! -d templates_c ]; then - echo " * Warning: templates_c didn't exist. I have created it, but you might want to change the ownership and reduce permissions" - # should really fix ownership to be that of the webserver; is there a nice way to discover which ? (www-data ?) mkdir -p templates_c && chmod 777 templates_c + + echo + echo " Warning: " + echo " templates_c directory didn't exist, now created." + echo + echo " You should change the ownership and reduce permissions on templates_c to 750. " + echo " The ownership needs to match the user used to execute PHP scripts, perhaps 'www-data' or 'httpd'" + echo + echo " e.g. chown www-data templates_c && chmod 750 templates_c" + echo fi echo echo "Please continue configuration / setup within your web browser. "