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
gshogi: init at 0.5.1 #41840
gshogi: init at 0.5.1 #41840
Conversation
@GrahamcOfBorg build gshogi |
Success on x86_64-linux (full log) Attempted: gshogi Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: gshogi Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: gshogi Partial log (click to expand)
|
pkgs/games/gshogi/default.nix
Outdated
buildPythonApplication rec { | ||
pname = "gshogi"; | ||
version = "0.5.1"; | ||
name = "${pname}-${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.
This line is not necessary for python apps since name
is derived automatically. Please delete.
meta = with stdenv.lib; { | ||
description = "A graphical implementation of the Shogi board game, also known as Japanese Chess"; | ||
homepage = http://johncheetham.com/projects/gshogi/; | ||
license = licenses.gpl3; |
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.
Doesn't build on darwin (the error looks like incompatible code), please restrict to linux by adding
platforms = platforms.linux;
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. Didn't have any darwin machines available, so I thought might as well try. Changed now.
pygobject3 | ||
pycairo | ||
]; | ||
|
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.
----------------------------------------------------------------------
Ran 0 tests in 0.000s
Since the package doesn't provide tests, disable them:
doCheck = false; # no tests available
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.
Oh, right. Added.
@GrahamcOfBorg build gshogi |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: gshogi Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: gshogi Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: gshogi Partial log (click to expand)
|
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.
Thank you!
* master: (161 commits) pcsclite: clean up after #41790 tor: 0.3.3.6 -> 0.3.3.7 opae: init at 1.0.0 tinc: 1.0.33 -> 10.0.34 tinc_pre: 1.1pre15 -> 1.1pre16 sit: 0.3.2 -> 0.4.0 (#41863) platforms/raspberrypi: enable kernelAutoModules libupnp: 1.6.21 -> 1.8.3 (#41684) androidStudioPackages.{dev,canary}: 3.2.0.16 -> 3.2.0.17 tdesktop: 1.3.0 -> 1.3.7 gns3Packages.{server,gui}{Stable,Preview}: 2.1.6 -> 2.1.7 aws-sam-cli: init at 0.3.0 (#41877) nixos/nat: optional networking.nat.externalInterface (#41864) linux: 4.17 -> 4.17.1 linux: 4.16.14 -> 4.16.15 linux: 4.14.48 -> 4.14.49 nixos/unbound: add restart (#41885) maintainers/create-azure.sh: remove hydra.nixos.org as binary cache (#41883) gshogi: init at 0.5.1 (#41840) neovim: add missing libiconv ...
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)