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

barony: init at 2.0.5 #25432

Closed
wants to merge 2 commits into from
Closed

barony: init at 2.0.5 #25432

wants to merge 2 commits into from

Conversation

lheckemann
Copy link
Member

Motivation for this change

Fun game.

While the engine is GPLv3, the game requires proprietary data to run. The path to this data can be specified with the -datadir=path/to/data option, or it can be run from the directory containing the data.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@joachifm
Copy link
Contributor

joachifm commented Jun 5, 2017

Why is the patch so large?

@lheckemann
Copy link
Member Author

Because it touches all the places where the game loads data files, which is all over the code. I've PRed the changes upstream, and they got accepted then reverted again because of some merging issues — they should be in the next release though and I'll remove the patch from the package once that happens.

@disassembler
Copy link
Member

@lheckemann do you think we should keep this PR open?

@lheckemann
Copy link
Member Author

Since upstream has now integrated my patch to allow setting a custom data dir, this can be made significantly smaller. However, not sure it should be integrated at all — it's got a fairly limited audience (just me, AFAIK) and I'm not sure additional stuff to maintain is what nixpkgs really needs…

@lheckemann lheckemann closed this Dec 29, 2017
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

4 participants