From 83dd99152bb8aa38d726d49769008f73e5b9a05d Mon Sep 17 00:00:00 2001
From: Simon Rettberg
Date: Fri, 9 Sep 2016 10:25:53 +0200
Subject: [SSUS] A function should do what its name says (plus the code was
 pointless anyways because it checked for existence the line before)

---
 satellit_upgrader/updater.template.sh | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

(limited to 'satellit_upgrader')

diff --git a/satellit_upgrader/updater.template.sh b/satellit_upgrader/updater.template.sh
index ebab457..93cafc3 100644
--- a/satellit_upgrader/updater.template.sh
+++ b/satellit_upgrader/updater.template.sh
@@ -68,10 +68,8 @@ restart_service () {
 }
 
 # ** Recursively set permissions of given directory to 0755/0644 optionally setting owner/group
-# ** If directory is not there it will be mkdir'ed.
 fixperms () {
 	[ -e "$1" ] || perror "normalize_perms: $1 does not exist"
-	[ -d "$1" ] || mkdir -p "$1"
 	chmod -R a-rw,u+rwX,go+rX "$1"
 	if [ -n "$2" ]; then
 		chown -R "$2" "$1" || pwarning "Could not change owner of $1 to $2"
@@ -231,7 +229,7 @@ if [ -n "$TGZ_SLXADMIN" ]; then
 	ln -s -f "$PATH_TFTP" "/srv/openslx/www/tftp" || perror "Could not symlink $PATH_TFTP to /srv/openslx/www/tftp"
 	echo "Web interface upgrade complete"
 	fixperms "${PATH_SLXADMINCACHE}" www-data:www-data
-	
+
 	if [ -h "${PATH_SLXADMINCACHE}/baseconfig-bwidm_settings.json" ]; then
 		pwarning "There is already a link or file /srv/openslx/www/slx-admin/modules-available/baseconfig_bwidm/baseconfig/settings.json. Please check manually, it should lead to /srv/openslx/www/slx-admin/modules-available/baseconfig_bwidm/baseconfig/settings.json"
 	else
-- 
cgit v1.2.3-55-g7522