Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove specific cinnamon condition #563

Merged
merged 4 commits into from
Jun 12, 2023
Merged

Conversation

mauricioszabo
Copy link
Contributor

Recently, I changed my desktop env from KDE to Cinnamon. Pulsar was not loading with the message:

╰─>$ pulsar .
/usr/bin/pulsar: line 139: ${$ACTION_PATH}: bad substitution

Honestly, I don't know what does lines do, and it seem flaky anyway, so this fixes the issue.

@Daeraxa
Copy link
Member

Daeraxa commented May 29, 2023

Where do you see this. I've been using it with cinnamon for ages and never seen this

@mauricioszabo
Copy link
Contributor Author

mauricioszabo commented May 29, 2023

@Daeraxa when trying to load pulsar from the command line - Ubuntu 22.04

@Daeraxa
Copy link
Member

Daeraxa commented May 29, 2023

The deb version? Same as I run but never seen it on mint 21

@Spiker985
Copy link
Member

Spiker985 commented May 29, 2023

Probably only a problem on a native nemo install

I believe it is used for the Open with Pulsar action

And this would be my bad, but should be able to be fixed simply by wrapping in double quotes

Edit: Would be fixed by removing the second $

Edit 2: The reason it's flaky, is because I didn't change the condition for matching to cinnamon, that was the way that the Atom team had matched it, rather than just checking that the value was literally the text they wanted. Might've been a sh -> bash thing.

And also that bit about the improper path for resources is also incorrect. If the file is included as part of our packaging, it would be correct

@mauricioszabo
Copy link
Contributor Author

@Spiker985 removing the second $ does indeed fixes the issue, but I always get the error:

cp: cannot create regular file '/usr/share/nemo/actions/pulsar.nemo_action': Permission denied

So it's not perfect, and I honestly fail to see how that could even work (copying a file to a root-protected folder without sudo)

@mauricioszabo mauricioszabo merged commit 11c91b4 into master Jun 12, 2023
@mauricioszabo mauricioszabo deleted the fix-issues-on-cinnamon branch June 12, 2023 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants