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
koules: init at 1.4 #108990
koules: init at 1.4 #108990
Conversation
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:
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
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.
Built on x86_64-linux. Runs fine. LGTM.
I marked this as stale due to inactivity. → More info |
This pull request has undergone review, I just force-pushed a new commit to adapt to the new |
pkgs/games/koules/default.nix
Outdated
''; | ||
|
||
preBuild = '' | ||
cp xkoules.6 xkoules.man |
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.
Why is this line needed, if the line on installPhase copies the original one?
Thank you @AndersonTorres for your extensive review! I believe to have resolved all the issues you highlighted. GitHub doesn't seem to allow me to comment directly on your question "Why is this line needed, if the line on installPhase copies the original one?" [Could not resolve to a node with the global id of 'MDE3OlB1bGxSZXF1ZXN0UmV2aWV3'.]. The answer is that because else the make phase fails (as alluded to in the comment of the build description). I wanted to keep the edits to the Makefile minimal, fixing the Makefile in this regard would also have been an option. Is it okay how it's done currently? :-) |
Okay for now. The comment explains it well enough. |
Motivation for this change
koules is an old and fun arcade game based on the fundamental law of body attraction. It is available in Debian and Arch. This pull request adds it to nixpkgs.
I went through path hell for installing this correctly. :-) Even with Debian's patches, which introduce
DESTDIR
, it took me some time to get everything working (in particular the soundserver, which is a separate binary).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)