From bb33e89ea3ce3acac54273afdd5537657d963ed6 Mon Sep 17 00:00:00 2001 From: Pavel Kiselev Date: Sun, 5 Nov 2017 11:29:56 +0300 Subject: [PATCH] Check if $id is non-zero You package non-statically compiled Node, so it doesn't work on systems which don't have glibc, e.g. Voidlinux. This causes $id to be an empty variable. This means `rm -r ~/.config/`. Yep, I just deleted my entire ~/.config directory by running your script. --- linux/uninstall.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/linux/uninstall.sh b/linux/uninstall.sh index bdccd06..e5b838e 100755 --- a/linux/uninstall.sh +++ b/linux/uninstall.sh @@ -19,7 +19,7 @@ rm ~/.config/chromium/NativeMessagingHosts/${id}.json echo " .. Removing manifest file for Mozilla Firefox" rm ~/.mozilla/native-messaging-hosts/${id}.json echo " .. Removing executable" -rm -r ~/.config/${id} +[ -n "$id" ] && rm -r ~/.config/${id} echo echo ">>> Native Client is removed <<<".