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
openjdk: download from mercurial repository to GitHub #102418
Conversation
2fbe61e
to
587ea38
Compare
Reasons: * openjdk is migrating to GitHub (https://openjdk.java.net/jeps/369) * GitHub provides faster download speed that mercurial
587ea38
to
aefda81
Compare
/marvin opt-in |
Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here. |
/cc @NeQuissimus |
Reminder: Please review! This Pull Request is awaiting review. If you are the assigned reviewer, please have a look. Try to find another reviewer if necessary. If you can't, please say so. If the status is not accurate, please change it. If nothing happens, this PR will be put back in the |
LGTM, but you should probably migrate the other openjdk versions (8, 12, 13, 14) as well. And then change the target branch to staging because that causes a lot of rebuilds. |
I'm going to add the other files to this PR. Only Java 8 development stayed in mercurial. There're mirrors by AdoptOpenjdk but I wonder if we should use this for building upstream openjdk package. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/hash-mismatch-building-jre-on-release-20-09/9898/11 |
I do see they have a tag for |
@basvandijk from my understanding the repository contains code produced during intial development of first java 8 release. So I don't think that tag you mentioned reflects the current state nixpkgs is shipping now. |
Reminder: Please review! This Pull Request is awaiting review. If you are the assigned reviewer, please have a look. Try to find another reviewer if necessary. If you can't, please say so. If the status is not accurate, please change it. If nothing happens, this PR will be put back in the |
I'd suggest to merge this change for now and postpone the decision if we should migrate the others repo too after that topic gains some more traction: https://discourse.nixos.org/t/retirement-of-old-openjdk-releases/9929 |
Can you backport to 20.09 too, please? |
openjdk: download from mercurial repository to GitHub (cherry picked from commit 01816a4)
Reasons:
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)