fix: same $HOME between install and uninstall (#784)

* fix: same $HOME between install and uninstall fixes issue https://github.com/datreeio/datree/issues/783. * PR comments: User facing log should still use ~ PR comments: User facing log should still use ~ instead of expanded variable. * unwrap apostrophe from here-document head label this enables parameterization. Also: escaping `\$id -u` so it is input literally. * revert noise

fix: same $HOME between install and uninstall (#784)
* fix: same $HOME between install and uninstall fixes issue https://github.com/datreeio/datree/issues/783. * PR comments: User facing log should still use ~ PR comments: User facing log should still use ~ instead of expanded variable. * unwrap apostrophe from here-document head label this enables parameterization. Also: escaping `\$id -u` so it is input literally. * revert noise
83527c68 · Netanel Haber · GitHub · b56f124c · 83527c68
Unverified Commit 83527c68 authored 2 years ago by Netanel Haber Committed by GitHub 2 years ago
Hide whitespace changes
Inline Side-by-side

Showing

with 2 additions and 2 deletions
+2 -2
--- a/install.sh
+++ b/install.sh
@@ -5,8 +5,8 @@ create_uninstall_script()
    UNINSTALL_SCRIPT="$HOME/.datree/uninstall.sh"
    touch $UNINSTALL_SCRIPT && chmod +x $UNINSTALL_SCRIPT

-    cat >> $UNINSTALL_SCRIPT << 'END'
-    if [ "$(id -u)" -ne 0 ] ; then
+    cat > $UNINSTALL_SCRIPT << END
+    if [ "\$(id -u)" -ne 0 ] ; then
    echo "This script must be executed with root privileges." && exit 1
    fi
    rm -f /usr/local/bin/datree