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
moon-buggy: init at 1.0.51 #28165
moon-buggy: init at 1.0.51 #28165
Conversation
pkgs/games/moon-buggy/default.nix
Outdated
}; | ||
|
||
meta = { | ||
inherit (s) version; |
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.
version
is not an valid attribute here.
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.
Thanks for the comments, I'll correct them. I was basing this off of the n2048 package, maybe I'll fix that one too while I'm at it.
pkgs/games/moon-buggy/default.nix
Outdated
inherit buildInputs; | ||
src = fetchurl { | ||
inherit (s) url sha256; | ||
}; |
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.
You can actually just inline all attributes instead of using inherit
. It does not really provide a benefit here. Just add rec
after mkDerivation
:
stdenv.mkDerivation rec {
pkgs/games/moon-buggy/default.nix
Outdated
inherit (s) version; | ||
description = ''A simple character graphics game where you drive some kind of car across the moon's surface''; | ||
license = stdenv.lib.licenses.gpl2; | ||
maintainers = []; |
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.
Do you want to maintain this package?
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.
Sure - I was wondering about this. Do I just add myself to lib/maintainers.nix and reference it in the list here?
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, thanks! Almost forgot this game from my childhood memories! 🍻
Motivation for this change
moon-buggy is fun
Things done
Please check what applies. Note that these are not hard requirements but merely serve as information for reviewers.
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)This is my first pull request. Please let me know anything I can improve on, because I'd like to contribute more in the future.