Skip to content
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

Merged
merged 1 commit into from Dec 24, 2018
Merged

Conversation

fusion809
Copy link
Contributor

Motivation for this change

OpenRA is out of date and it needs to be updated, simple as that.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

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).

@fusion809
Copy link
Contributor Author

fusion809 commented Dec 19, 2018

Let us /CC @rardiol, the maintainer.

@rardiol
Copy link
Contributor

rardiol commented Dec 23, 2018

Sorry for taking so long, seems fine to me ( but I didn't test multiplayer either).

@@ -5,7 +5,7 @@

stdenv.mkDerivation rec {
name = "openra-${version}";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
name = "openra-${version}";
pname = "openra";

Copy link
Contributor Author

@fusion809 fusion809 Dec 23, 2018

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@Mic92
Copy link
Member

Mic92 commented Dec 23, 2018

@GrahamcOfBorg build openra

@fusion809
Copy link
Contributor Author

OK, well all checks have passed, so whom wishes to give their tick of approval and perhaps even merge this PR?

@veprbl veprbl merged commit 9592e38 into NixOS:master Dec 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants