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
openra: 20180923->20181215 #52531
openra: 20180923->20181215 #52531
Conversation
Let us /CC @rardiol, the maintainer. |
Sorry for taking so long, seems fine to me ( but I didn't test multiplayer either). |
pkgs/games/openra/default.nix
Outdated
@@ -5,7 +5,7 @@ | |||
|
|||
stdenv.mkDerivation rec { | |||
name = "openra-${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.
name = "openra-${version}"; | |
pname = "openra"; |
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.
OK, although this has nothing to do with the topic of this PR, nor my original commit, I didn't change this line, after all. But, I suppose might as well incorporate as many minor improvements as possible in this PR while we're at it.
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.
f0576c8
to
2df9920
Compare
@GrahamcOfBorg build openra |
OK, well all checks have passed, so whom wishes to give their tick of approval and perhaps even merge this PR? |
Motivation for this change
OpenRA is out of date and it needs to be updated, simple as that.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)I have tried out all three mods and all three work fine, at least in single player mode (haven't tried multiplayer as I have never played a multiplayer game).