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
minecraft: 2015-07-24 -> 2.1.5965 (closes #68123) #68175
Conversation
7419c8f
to
9477f97
Compare
We can't we use the
url? |
The flatpak uses that URL I recommend just fetching the flathub repo and installing their desktop integrations. |
9477f97
to
10b65cf
Compare
Thank you for the review! |
10b65cf
to
1816011
Compare
1816011
to
49f5314
Compare
To use the tar.gz url, since there's only a single icon (that probably won't ever change), we can just embed it as nixpkgs/pkgs/games/minecraft/minecraft-launcher.svg, and for the desktop item we can use |
I think this json file has links to the launcher files I see no mention of a permalink for a debian file. Perhaps looking at an ubuntu ppa could yield some info. I've also looked at all references to minecraft launcher on repology and they either use that link to the debian file, the one I've mentioned, or completely vendor the binary. I don't think we technically lose anything from fetching the files from flathub, it would be much worse to add them in tree. |
That JSON URL seems pretty useful to write an update script! What we do lose by fetching from flathub is the ability for the build to succeed even if flathub doesn't provide those files anymore. |
We don't distribute this. And it's a git repo, only thing you'd worry about is it being completely deleted. |
See #68123 (comment). We should switch the launchers license from For the icon, @peterix indicated an offer to put it at an official url for grabs. |
Is this just about the official minecraft icon? If it's the same as the website, you could always fetch the archived version from today by archive.org. IANAL but I think their archiving and us fetching said archive is both perfectly legal (as long as we don't redistribute). |
Unfortunately not, we need the transparency. But it looks like we'll get a fixed path to the icon or something that includes it. |
49f5314
to
2fc5ce9
Compare
@infinisil are you still waiting for upstream, for the icon? |
That would be nice yeah, I'm not sure if there's a legally correct way to get the icon without it |
Here you go:
|
Thanks a lot! ❤️ |
2fc5ce9
to
a0087cd
Compare
switched to the new official launcher, renamed to minecraft-launcher, and added an update script
a0087cd
to
3a635da
Compare
Thanks a lot @peterix! |
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.
Looking good
Backported to 19.09 in 10903f5 |
Is 19.03 affected? |
Yeah, I'll backport to 19.03 too |
Backported to 19.03 in 7086c81 |
minecraft: 2015-07-24 -> 2.1.5965 (closes NixOS#68123) (cherry picked from commit c545e85)
Motivation for this change
Closes #68123.
Opened in WIP because its 'final form' is blocked on external input and so people can already use it and packaging effort isn't duplicated.
Things to do
libfliteWrapper.so
)Things done
sandbox
innix.conf
on non-NixOS)./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @cpages @ryantm @infinisil