h3n3/kiauh
1
0
Fork 0
Code Issues 46 Pull Requests 10 Packages Projects Releases Wiki Activity

Add KlipperScreen to kiauh #44

Merged
jordanruthe merged 3 commits from master into master 2020-11-28 16:45:48 +01:00
Conversation 4 Commits 3 Files Changed 12 +2 -2
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 622e8251a3 - Show all commits

2
scripts/remove.sh
View File

@@ -321,7 +321,7 @@ remove_klipperscreen(){
rm -rf $KLIPPERSCREEN_ENV_DIR && ok_msg "Directory removed!"
fi
#remove KlipperScreen systemd file
if [ -e /etc/nginx/sites-available/mainsail ]; then
if [ -e /etc/systemd/system/KlipperScreen.service ]; then
status_msg "Removing KlipperScreen configuration for Nginx ..."
dw-0 commented 2020-11-28 09:26:34 +01:00 (Migrated from github.com)
Review
Copy Link

The if condition here looks like a typo.
You probably wanted to check for [ -e /etc/systemd/system/KlipperScreen.service ].

And the status_msg does not suit the action followed by it.
You probably want to call it status_msg "Removing KlipperScreen Service ..."

The if condition here looks like a typo. You probably wanted to check for `[ -e /etc/systemd/system/KlipperScreen.service ]`. And the status_msg does not suit the action followed by it. You probably want to call it `status_msg "Removing KlipperScreen Service ..."`
sudo rm /etc/systemd/system/KlipperScreen.service && ok_msg "File removed!"
fi

2
scripts/update.sh
View File

@@ -184,5 +184,5 @@ update_klipperscreen(){
ok_msg "Dependencies have been installed!"
fi
ok_msg "Update complete!"
start_klipper
start_klipperscreen
}