-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Please stop using absolute paths for desktop files. #1755
Comments
I understand this might complicate matters for Signal, but it's acceptable for even |
#!/bin/bash
set -e
exec /opt/Signal/signal-desktop "$@" |
[Desktop Entry]
Name=Signal
Comment=Private messaging from your desktop
Exec=signal-desktop %U
Terminal=false
Type=Application
Icon=signal-desktop
StartupWMClass=Signal
Categories=Network; |
The package already creates a symlink from |
On as side note from the initial issue topic. I don't believe /usr/local/bin is the correct location to store a symlink. See here for what should go in /usr/bin, and here for what should go in /usr/local/bin. I think a better place for the symlink is in /usr/bin over /usr/local/bin. As some programs might not identify signal-desktop as being installed. (e.g. How I ended up here) |
Please, can we switch to installing the symlink in |
Bug description
Signal Desktop uses an absolute path for it's desktop file located at
/usr/share/applications/signal-desktop.desktop
, which is contrary to what you should be doing to maintain an extensible and customizable system. This prevents people from wrapping on-top of Signal to prefer pre-tasks or setup.Steps to reproduce
Screenshots
Platform info
Browser: Non-relevant.
Operating System: Ubuntu 16.04, 17.10
Signal version: Non-Relevant.
On Linux it is preferred that you use a non-absolute searchable binary-name, so that the system can decide based on the users preference what file to use to launch. Using an absolute path prevents me from creating a wrapper that forces Signal into a slice, or onto a Docker image on the fly or anything that would require me to create a system-wide wrapper.
On our systems it would be desireable to be able to wrap Signal to prevent it from using resources we don't want it to use. We can force Systemd to reclaim memory we don't want Signal to have. Lets face it, you are using JavaScript on the desktop, inside of a Chrome window, using v8, and while we accept this is the easiest way for you to ship us a cross-platform application, we do not accept the amount of resources that Chrome opportunistically claims in excess of reasonable.
The text was updated successfully, but these errors were encountered: