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
frozen-bubble: install desktop file #86914
Conversation
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.
Awesome! I use a DE without desktop entry support, so I missed this in my time porting it. There's two small issues I'd like to see resolved before approving, see comments.
I marked this as stale due to inactivity. → More info |
@iblech friendly ping |
Thank you @SuperSandro2000 for the friendly ping, and thank you @puckipedia and @teto for your reviews! Very much appreciated. I don't have time to incorporate your suggestions in the next couple of days, but want to do so soon. If you want to beat me to it, then feel free, but no worries, else I'll do it :-) |
@SuperSandro2000 I finally got around to this -- I believe this pull request can be merged now. :-) |
a85e98d
to
db28ff0
Compare
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 1 package built:
|
I'm closing this because I just noticed that @khumba has pushed a commit with the same effect :-) |
Hi @iblech, I'm sorry I didn't search and see this PR before I opened mine! This one is better, it would be nice to have the icons too. I'll do a proper search and not just check |
@khumba Absolutely no worries, I'm happy that we now have the desktop file. In the past I also overlooked already existing pull requests. It's bound to happen with 2.4k open :-) |
Motivation for this change
Thank you, @puckipedia, for maintaining Frozen Bubble in nixpkgs! This pull request just adds a desktop file (lifted from Debian) together with appropriate icons (which are already part of the upstream package).
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)