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
bonzomatic: init at 2018-03-29 #39171
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.
Built and tested this locally, works without problems. It need a simple config.json
file in the current working directory to get started though. Here's one:
{
"window":{
"width":1440,
"height":900,
"fullscreen":false
},
"font":{
"file":"/nix/store/h17zrpn0zfgx0p65ic457kb9plwll3n0-nerdfonts-1.1.0/share/fonts/truetype/NerdFonts/Iosevka Nerd Font Complete.ttf",
}
}
(I got the font path with fc-list
)
@GrahamcOfBorg build bonzomatic |
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: bonzomatic Partial log (click to expand)
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: bonzomatic Partial log (click to expand)
|
let | ||
name = "bonzomatic"; | ||
in stdenv.mkDerivation { | ||
inherit name; |
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.
Please add a version in this package (the version is used by nix-env to upgarde). Something such as:
name = "${pname}-${version}"
pname = "bonzomatic";
version = "2018-03-29";
{ stdenv, makeWrapper, fetchFromGitHub, cmake, alsaLib, mesa_glu, libXcursor, libXinerama, libXrandr, xorgserver }: | ||
|
||
let | ||
name = "bonzomatic"; |
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.
By using a recursive attribute set as derivation argument, this is not required. You can see the rec
keyword.
Thanks for the review @nlewo and @infinisil
It would be nice to provide a default configuration file since it is not trivial to create it. This would be useful for reviewers to test future updates. This could be addressed in another PR. |
Motivation for this change
Adds Bonzomatic, a live-coding tool for writing 2D fragment/pixel shaders.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)I added the unfreeRedistributable license, as the upstream repo contains libbass.so which is only free for non-commercial use.