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
fish-fillets-ng: install desktop file #86912
Conversation
I marked this as stale due to inactivity. → More info |
I'm closing as op is not responsive. |
Sorry for being not responsive! I will implement the changes suggested by the review in the next couple of days and then reopen this pull request. :-) |
4ece4bd
to
289c46d
Compare
@doronbehar I finally got around to tending to this pull request and believe that it can now be merged. Could you have a new look? |
Thank you @SuperSandro2000 for your review, I believe I fixed all issues you mentioned. :-) |
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.
other than that diff LGTM
Fixed according to diff. :-) |
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:
|
Awesome, thank you both, @SuperSandro2000 and @7c6f434c for reviewing and merging! |
Motivation for this change
Thank you, @raskin, for your work maintaining this game in nixpkgs!
This pull requests adds the desktop file (lifted from Debian) so that the game is easily reachable from graphical menus.
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)