Merge pull request #563 from pulsar-edit/fix-issues-on-cinnamon

CHANGELOG.md

@@ -6,6 +6,8 @@
 
 ## [Unreleased]
 
+- Fixed bug that happens on some systems when trying to launch Pulsar using the Cinnamon desktop environment
+- Added a modern implementation of Tree-sitter grammars behind an experimental flag. Enable the “Use Modern Tree-Sitter Implementation” in the Core settings to try it out
 - Bugfix: fixed Clojure indentation on tree-sitter
 - Improved the Clojure language support by migrating it to tree-sitter and support block comments, quoting, and other advanced features on modern tree-sitter implementation
 - Added a modern implementation of Tree-sitter grammars behind an experimental flag. Enable the “Use Modern Tree-Sitter Implementation” in the Core settings to try it out.

pulsar.sh

@@ -125,21 +125,6 @@ elif [ $OS == 'Linux' ]; then
 
   PULSAR_PATH="/opt/Pulsar/pulsar"
 
-  #Will allow user to get context menu on cinnamon desktop enviroment
-  #Add a check to make sure that DESKTOP_SESSION is set before attempting to grep it
-  #expr substr is expecting 3 arguments string, index, length
-  #If grep doesnt find anything is provides an empty string which causes the expr: syntax error: missing argument after '8' error - see pulsar-edit/pulsar#174
-  #Im also not quite sure why they used grep instead of simply [ "${DESKTOP_SESSION}" == "cinnamon" ]
-  if [ -n "${DESKTOP_SESSION}" ] && [ "$(expr substr $(printenv | grep 'DESKTOP_SESSION=') 17 8)" == "cinnamon" ]; then
-    #This local path is almost assuredly wrong as it shouldnt exist in a standard install
-    ACTION_PATH="resources/linux/desktopenviroment/cinnamon/pulsar.nemo_action"
-
-    #Validate the file exists before attempting to copy it
-    if [ -f "${ACTION_PATH}" ]; then
-        cp "${$ACTION_PATH}" "/usr/share/nemo/actions/pulsar.nemo_action"
-    fi
-  fi
-
   #Set tmpdir only if tmpdir is unset
   : ${TMPDIR:=/tmp}