-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
r2modman: init at 3.1.42 #229551
r2modman: init at 3.1.42 #229551
Conversation
Is there a good way to add the steam libraries so it can properly launch modded steam games? |
I think you would need to run it inside the Steam FHS environment. I think letting the user opt to launch (I've just been using this to install mods, and then launch the game from Steam.) |
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 left a bunch of review comments, but this looks pretty good! I do wonder if it would benefit from organizing it without mkYarnPackage
like how the teams-for-linux
derivation does now (which is also an electron app that uses yarn)
e9d415e
to
f8d6d9b
Compare
That package does look a bit tidier. I will attempt to redo this package following that pattern, and for heroic as well if that works out. |
Hopefully we can get back to #210814 soon and have a modern builder instead of mixing and matching yarn2nix-moretea and |
3220985
to
298f584
Compare
I have redone the package using |
52a4d9d
to
105fc5e
Compare
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.
Looks good to me, only thing remaining is launching steam games from the app. For general use I do launch via steam itself, not r2modman, but it would be helpful for user-friendliness. I can't think of a better way than maybe adding an option to derivation to use steam-run, even though that still feels awkward.
I think we should address supporting |
Sounds good, I like the idea of having an option to set a launch wrapper, kinda like how Prism Launcher does it. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
pkgs/games/r2modman/default.nix
Outdated
(cd public/icons | ||
for img in *png; do | ||
dimensions=''${img#favicon-} | ||
dimensions=''${dimensions%.png} | ||
mkdir -p $out/share/icons/hicolor/$dimensions/apps | ||
cp $img $out/share/icons/hicolor/$dimensions/apps/${pname}.png | ||
done) |
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.
(cd public/icons | |
for img in *png; do | |
dimensions=''${img#favicon-} | |
dimensions=''${dimensions%.png} | |
mkdir -p $out/share/icons/hicolor/$dimensions/apps | |
cp $img $out/share/icons/hicolor/$dimensions/apps/${pname}.png | |
done) | |
cd public/icons | |
for img in *png; do | |
dimensions=''${img#favicon-} | |
dimensions=''${dimensions%.png} | |
mkdir -p $out/share/icons/hicolor/$dimensions/apps | |
cp $img $out/share/icons/hicolor/$dimensions/apps/${pname}.png | |
done | |
cd ../.. |
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 much prefer cd
in a subshell in shell scripts, so I would like to keep this bit as it is if that's OK.
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.
Fixed.
9938b91
to
87b3900
Compare
@huantianad Do you want to be put down as a maintainer on this? |
Yeah sure, I'm down to help maintain. |
Description of changes
Add r2modman, an unofficial Thunderstore mod manager.
Builds from source, superseding PR #201708.
Closes #143659
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)