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
plex-media-player: remove custom desktopItem #60444
plex-media-player: remove custom desktopItem #60444
Conversation
I have tested this and can confirm it builds and had two desktop items before and now has 1. I have not tested any binaries |
Great! The upstream desktop file is more fully featured anyways :) |
Is installing the icon necessary? I would expect them to install it on their own when they added a desktop file. Are the paths in the desktop file correct? Sometimes project hardcode |
Indeed, the install part is not needed, the icon is fine even after removing that part. There is no path in the desktop file: stanislas@nixpsla ~/nixpkgs> cat result/share/applications/plexmediaplayer.desktop
[Desktop Entry]
Version=1.0
Name=Plex Media Player
GenericName=Media Player
Comment=View your media
Exec=plexmediaplayer --fullscreen --tv
Icon=plexmediaplayer
Terminal=false
Type=Application
Categories=AudioVideo;Video;Player;TV;
Actions=TV;DesktopF;DesktopW;
[Desktop Action TV]
Name=TV
Exec=plexmediaplayer --fullscreen --tv
[Desktop Action DesktopF]
Name=Desktop [Fullscreen]
Exec=plexmediaplayer --fullscreen --desktop
[Desktop Action DesktopW]
Name=Desktop [Windowed]
Exec=plexmediaplayer --windowed --desktop |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested the desktop item and all icons and actions work.
Please squash your commits accordingly and we should be good to merge 😄
one to fix desktopItem
, another to not copy icon (or whatever)
ed942fc
to
5f1dc02
Compare
It should be good now, I hope 🙂 |
✨ Thank you @angristan for contributing |
Thanks everyone for helping out! 🙇♂️ |
Motivation for this change
Fixes #60437:
plex-media-player
comes with a working desktop file, so there is no need to add our own custom one unless we remove the original. Both seem to work fine, so let's keep it simple.Things done
makeDesktopItem
dependencydesktopItem
(plex-media-player.desktop
)sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)