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
xmage: init at 1.4.42V6 #82655
xmage: init at 1.4.42V6 #82655
Conversation
The server does not yet have a script to start. |
The derivation lacks meta, and notably the license. |
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.
otherwise LGTM
opens fine
[3 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/82655
1 package built:
xmage
url = "https://github.com/magefree/mage/releases/download/xmage_1.4.42V6/xmage_${version}.zip"; | ||
sha256 = "14s4885ldi0rplqmab5m775plsqmmm0m89j402caiqm2q9mzvkhd"; | ||
}; | ||
|
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.
# since we are just fetching and wrapping, don't have remote jobs execute this | |
preferLocalBuild = true; |
description = "Magic Another Game Engine"; | ||
license = licenses.mit; | ||
maintainers = with maintainers; [ matthiasbeyer ]; |
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.
add homepage please
squashed and committed |
The xmage script added during the build doesn't work, because xmage tries to write to its installation directory (card images downloaded from the internet are stored alongside the installed plugins). I dug some into xmage but it seems it's fairly deeply hardcoded to write to the same directories it reads plugins from. I opened magefree/mage#7025 about this upstream. |
Ideally they should have some logic like, |
Motivation for this change
Corona
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)