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
openmsx: init at git-2017-11-02 #31184
Conversation
stdenv.mkDerivation rec { | ||
|
||
name = "openmsx-${version}"; | ||
version = "git-2017-11-02"; |
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 not 0.14.0? (https://github.com/openMSX/openMSX/releases/tag/RELEASE_0_14_0)
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.
I have talked with the openMSX developers in their IRC channel, and after presenting a bit of NixOS to them, they added a simple patch to make it build under Nix.
Do you think using the patch file is better than the hardcoded git?
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.
ping @yegortimoshenko
OpenMSX is an emulator for the MSX home computer system. Its goal is | ||
to emulate all aspects of the MSX with 100% accuracy. | ||
''; | ||
homepage = http://openmsx.org; |
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.
Nitpick: HTTPS version is available too
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.
Done!
customBuildFile = ./custom-nixos.mk; | ||
|
||
postUnpack = '' | ||
cp ${customBuildFile} $sourceRoot/build/custom.mk |
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.
Semantically this fits better in postPatch
, additionally making $sourceRoot
unnecessary. Also customBuildFile
can be inlined:
postPatch = ''
cp ${./custom-nixos.mk} build/custom.mk
'';
Motivation for this change
New package - openmsx
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)