On Fri, 17 Jan 2025 18:38:25 GMT, Eirik Bjørsnøs <eir...@openjdk.org> wrote:
>> Please review this cleanup PR which updates a total of 12 links to external >> documentation or references in `java.base` to use https instead of plain >> text http. >> >> Links in `java.security` and `share/data/tzdata` are excluded from this PR. >> >> This is a documentaton-only cleanup. No tests are added or updated. >> `noreg-cleanup` added in the JBS. All updated links have been verified to >> resolve and to show the expected content. >> >> There are two files here with non-Oracle copyright headers, not quite sure >> how to update those? (`java_md_aix.h` and `xss-common-qsort.h`). >> >> Changes beyond the obvious are as follows: >> >> `linux/native/libsimdsort/xss-common-qsort.h`: >> The current link uses a non-default port number. Changed to using the >> default port number. Reported upstream as intel/x86-simd-sort#170 >> >> `share/man/keytool.1`: >> www.oracle.com redirects this from >> `/technetwork/java/javase/javasecarootcertsprogram-1876540.html` to >> `/java/technologies/javase/carootcertsprogram.html`. Using the new URL here. >> Should this be updated in Oracle-internal sources as well? >> >> `unix/classes/sun/net/PortConfig.java` >> The current link no longer resolves. As a replacement, I suggest >> https://www.ibm.com/support/pages/node/886227 which answers a support >> question regarding ephemeral ports. >> >> `unix/native/libjava/*`, >> Links redirect from "www.opengroup.org" to "pubs.opengroup.org". Using this >> new host in the URLs >> >> `unix/native/libjava/ProcessImpl_md.c` >> A link to pasc.org now requires authentication. I changed this to be a >> Wayback link. Alternantives would be to delete this link, or to track down a >> revised version of the spec the linked Interpretation Request refers to. > > Eirik Bjørsnøs has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains seven commits: > > - Merge branch 'master' into links-use-https > - Re-fix link to "Oracle Java Root Certificate program" in keytool.md > - Merge branch 'master' into links-use-https > > # Conflicts: > # src/java.base/share/man/keytool.1 > - Remove accidental "'" > - Use Wayback link > - Update copyright year > - Update links in java.base to use https:// @JoKern65 Can you have a look at the AIX link? ------------- PR Comment: https://git.openjdk.org/jdk/pull/21633#issuecomment-2602830677
