-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
modrinth-app: init at 0.7.1 #289149
modrinth-app: init at 0.7.1 #289149
Conversation
almost forgot: i'm not really sure how we are meant to handle wrappers (a la prismlauncher) in the new by-name scheme. let me know if my guess was wrong :p |
44886d7
to
0d8f4c3
Compare
Thanks for doing this! Result of 2 packages built:
|
0d8f4c3
to
5266984
Compare
5266984
to
76a3abb
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3654 |
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.
code looks good; ofborg is happy; Alanazane can successfully play Minecraft with it 👍
76a3abb
to
304ae01
Compare
304ae01
to
5c5762e
Compare
fixed building on darwin |
5c5762e
to
abdd95c
Compare
abdd95c
to
8263550
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3881 |
When rebased onto master, this fails to build because of #307077 |
8263550
to
d043810
Compare
thanks for catching this :) |
Description of changes
the modrinth app is a unique, open source minecraft launcher that allows you to play your favorite mods, and keep them up to date, all in one neat little package
it is marked as
unfreeRedistributable
due to the assets being used. it can also compile on aarch64, however the launcher itself doesn't support it yetfixes #247468
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.