WhyNotHugo / caffeine-ng

⚠ This project has migrated to codeberg.org
https://codeberg.org/WhyNotHugo/caffeine-ng
GNU General Public License v3.0
180 stars 21 forks source link

Desktop file should not include `/usr/bin/` #88

Closed WhyNotHugo closed 2 years ago

WhyNotHugo commented 2 years ago

It should be assumed that caffeine is in $PATH.

This facilitates setting up things like firejail.

pietryszak commented 2 years ago

When I use manual install, caffeine is installed in /usr/local/bin/, so desktop file has wrong path /usr/bin/.

After change in .desktop to exec=caffeine everything works fine.

But in caffeine-ng properties exec=caffeine -p not working because that option not exist /usr/local/bin/caffeine -p Usage: caffeine [OPTIONS] COMMAND [ARGS]... Try 'caffeine --help' for help. Error: no such option: -p

Bug confirmed.

WhyNotHugo commented 2 years ago

Leaving this open since the preferences desktop file is still broken.

pietryszak commented 2 years ago

Leaving this open since the preferences desktop file is still broken.

@WhyNotHugo Caffeine desktop file working fine, but preferences desktop not. To be honest, preferences desktop file is unnecessary in my opinion.