-
-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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 15.0.2 #69460
openjdk 15.0.2 #69460
Conversation
Patches look like they're no longer necessary: They also fail to apply: https://github.com/Homebrew/homebrew-core/runs/1739892097#step:7:63 |
All three Intel CI nodes:
These are the contents of
Looks like the build number is no longer available here. homebrew-core/Formula/openjdk.rb Lines 106 to 111 in f9b4c5b
|
For openjdk 15.0.1, this is the content of
This gives the build number
|
Why do we need to have the build number? If it's useful I guess we could move to the arm way (since that's where they seem to be going) |
No idea.
That would've given a build number of 8 instead of 9 for 15.0.1 though: #69460 (comment) Not sure if that's important at all. |
It "should" be +8. Usually they have two tags pointing to a single commit like so:
for some reason upstream has not done it this time. We could potentially wait for adoptopenjdk to release 15.0.2 to see what build number they pick. |
Let's just go with this since it's easier. I think build numbers are really only used when comparing among the same java patch version and I think it's a legacy of openjdk 8-era java versioning (like jdk8u282) |
Done. This is still slightly different from ARM, which uses only the major version, though. However, I assume that's because |
Same error; still can't find the build number. (I forgot to change the error message to
Ah, I see. I forgot to remove the |
2c8baf8
to
9bdad7e
Compare
Re the build number: #68946 (comment) It seems we should add one to it on Intel? (I'm not even sure how important this is.) |
ARM CI broken by Homebrew/brew#10375.
|
Big Sur
Catalina
Mojave
|
Looks like adoptopenjdk is using 15.0.2+7 🤷 |
9bdad7e
to
3f9024b
Compare
3f9024b
to
1ba9cee
Compare
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
1ba9cee
to
c1dfb99
Compare
mesos and man-db are now fixed, so hopefully this CI run works. |
Mojave is 🍏 🙏 |
Big Sur:
Catalina:
A little surprised the ARM runner is still going. |
ARM runner is still going. It seems to be weird to have to wait for it, given that this PR should have no impact on the ARM bottle other than renaming it... |
I'm just going to publish these bottles. OpenJDK 16 is coming out in a few weeks so we need to direct our energy there 🤷 |
I logged into the ARM runner and determined that an automatic update (of the runner) seemed to have killed the runner instance, which is why the tests were stalled. |
Created with
brew bump-formula-pr
.resource
blocks may require updates.