Clarify requirements for AIX runtimes #2085
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Deploy Preview for eclipsefdn-adoptium ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov Report
@@ Coverage Diff @@
## main #2085 +/- ##
=======================================
Coverage 99.16% 99.16%
=======================================
Files 85 85
Lines 6199 6199
Branches 509 509
=======================================
Hits 6147 6147
Misses 52 52 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Signed-off-by: Stewart X Addison <[email protected]>
gdams
approved these changes
Aug 11, 2023
tellison
requested changes
Aug 11, 2023
| @@ -87,10 +87,8 @@ icon:check[] - Supported, icon:docker[] - Docker image available, icon:times[] - | |||
| | Solaris 11 | icon:check[] | icon:times[] | icon:times[] | icon:times[] | |||
| | Solaris 10u11 | icon:check[] | icon:times[] | icon:times[] | icon:times[] | |||
|
|
|||
| 5+h| AIX (PowerPC 64-bit Big Endian) | |||
| 5+h| AIX (PowerPC 64-bit Big Endian) footnote:aix71[Running the latest versions on AIX 7.1 is no longer supported. The last versions which targetted able to run on AIX 7.1 were jdk8u362, jdk-11.0.18 and jdk-17.0.8] footnote:alxrt[Requires XLC runtime packages from the compiler or standalone for XLC 13 for JDK8: https://www.ibm.com/support/pages/ibm-xl-cc-runtime-aix-131 or XLC 16 for JDK11+: https://www.ibm.com/support/pages/ibm-xl-cc-runtime-aix-161] | |||
There was a problem hiding this comment.
Combine into one footnote, and use linked text rather than display URLs.
Suggested change
| 5+h| AIX (PowerPC 64-bit Big Endian) footnote:aix71[Running the latest versions on AIX 7.1 is no longer supported. The last versions which targetted able to run on AIX 7.1 were jdk8u362, jdk-11.0.18 and jdk-17.0.8] footnote:alxrt[Requires XLC runtime packages from the compiler or standalone for XLC 13 for JDK8: https://www.ibm.com/support/pages/ibm-xl-cc-runtime-aix-131 or XLC 16 for JDK11+: https://www.ibm.com/support/pages/ibm-xl-cc-runtime-aix-161] | |
| 5+h| AIX (PowerPC 64-bit Big Endian) footnote:aix71[Running the latest versions on AIX 7.1 is no longer supported. The last versions able to run on AIX 7.1 were jdk8u362, jdk-11.0.18 and jdk-17.0.8. Later versions require XLC runtime packages from the compiler or standalone for link:https://www.ibm.com/support/pages/ibm-xl-cc-runtime-aix-131[XLC 13] for JDK8 or link:https://www.ibm.com/support/pages/ibm-xl-cc-runtime-aix-161[XLC 16] for JDK11+. ] |
There was a problem hiding this comment.
I will move my change requests to a new PR
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Remove AIX 7.1 with a note referencing the last versions that worked on that level.
Provide links to the AIX XLC runtime downloads which are prerequisites for running Temurin on AIX.
This infromation was spotted as missing from this slack conversation
Checklist
npm testpasses