-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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
Added new Game pkg for War Thunder, an MMO tank/plane/warship simulator with a native linux client. #372991
base: nixos-24.11
Are you sure you want to change the base?
Conversation
Fuck yeah
…her runs on boot, but spawns aces child when the download completes. Game errors out with VK initialization. This *should* fix that.
You seriously need to rebase / squash those commits into one. Nobody wants to read this commit history of 89 commits with swearwords in every 3rd or so. Please, also go through https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md. This needs to target master, needs proper commit + PR titles etc. Several CI jobs are failing, giving you feedback on what needs to be improved as well: You need to run nixfmt, put this in I suggest you start reading up on the contributing guide and fix as much as you can. |
|
||
if [ ! -d "$HOME/.aces64/War-Thunder-086d99e" ]; then | ||
|
||
echo "DEBUG::: Directory not found. Fuck." |
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.
Stuff like this is not acceptable.
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.
I totally forgot to fix these, i used them as markers mostly during silent fails. Ill fix.
"rebase / squash those commits into one"
Im not sure how to do that just yet as im new to contributing but i will figure it out and do that as well, and read the doc for contributing you provided and fix that as well.
Should i close this merge request and make a new one, or how should i approach?
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.
Should i close this merge request and make a new one, or how should i approach?
No, you don't need to close it. Rebasing / squashing is what you want to solve that - this allows you to fix everything in here.
…w. Will fix later, added to TODO
This build is UNTESTED on linux with nix but is tested on nixOS, will do testing on nix pkg manager with other distros but it was written with other distros in mind so it SHOULD work.
This build is fully tested and (to the best of my ability as a solo) debugged, this version runs the game nearly twice as fast as the linux or proton/windows version available through steam with no crashing bugs getting 175 average frames relative to the 75 frames at the same setting through the steam port.
Check: https://www.protondb.com/app/236390 if your curious.
The game includes a few binary applications, notably acesx86_64 as the wrapper, and the launcher.
This package NEEDS write access to the users home directory BUT is designed to be installed as a system level package. The wrapper sets users $HOME/.aces64 directory as the execution point so the launcher can download the full game. This preserves the immutability of the nix store while also preserving game data that complies with the games anti-cheat and runtime env variables pre-compiled into the original binary.
It also installs a desktopItem file so the user can start the wrapper from the desktop as a standard application like Lord Gabe intended, or add it the steam library as an executable where steam can launch it natively if the user needs to run a comp like Gamescope as a startup option since gamescope is not included in this derivation due to env variable bugs i didnt want to deal with and ultimately, were unnecessary.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.