-
Notifications
You must be signed in to change notification settings - Fork 57
8278274: Update nroff pages in JDK 19 before RC #145
Conversation
👋 Welcome back dholmes! A progress list of the required criteria for merging this PR into |
@dholmes-ora The following labels will be automatically applied to this pull request:
When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding mailing lists. If you would like to change these labels, use the /label pull request command. |
/label remove kulla |
@dholmes-ora |
/label remove hotspot-jfr |
@dholmes-ora |
@dholmes-ora |
@@ -453,15 +453,15 @@ specified. | |||
The \f[CB]\-signer\f[R] value specifies the alias of a | |||
\f[CB]PrivateKeyEntry\f[R] for the signer that already exists in the | |||
keystore. | |||
This option is used to sign the certificate with the signer???s private | |||
This option is used to sign the certificate with the signer?s private |
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.
Not a problem with this PR as such, but we still have a ?
character in the output.
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.
Yeah I spotted that too, but it would need to be fixed in source and nroff. Must be some kind of "smart quote" from an editor. Do you think this needs to be fixed or just handle it in mainline?
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.
Filed JDK-8290626. It can easily be fixed before RDP2.
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.
Hello @jonathan-gibbons
Excuse my hijacking / piggy-backing on this conversation!
When you say
Not a problem with this PR as such, but we still have a
?
character in the output.
This strongly suggests that there must be an input to those man pages somewhere. :-) Would you mind to point me to it? Are they even included in the git repo?
Also, I was wondering why these man pages (i.e. the outputs) are obviously not included any more in the OpenJDK builds, e.g. https://jdk.java.net/19/ ? I am not sure when they were removed or if they were ever included in _open_jdk builds at all, but I see them in the Oracle JDK 1.8 Build.
I find them really nice to read! Are there other output formats, e.g. HTML, that might be are deployed somewhere on the web?
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.
It did not take much effort to answer the last of the above question myself.
Are there other output formats, e.g. HTML, that might be are deployed somewhere on the web?
Obviously "yes". HTML renderings can be found by following the, e.g. "Java Development Kit Version 18 Tool Specifications" Link on the "Specifications Overview" page at, e.g. "https://docs.oracle.com/en/java/javase/18/docs/specs/".
Still hunting for the sources.
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.
The version changes in each file look good (19-ea
to 19
).
The changes for javadoc look good.
I looked over the other changes for other files, and while they look good, I cannot speak for their technical accuracy. That being said, this is an automated process deriving info from upstream, so is likely OK.
@dholmes-ora This change now passes all automated pre-integration checks. ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details. After integration, the commit message for the final commit will be:
You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed. At the time when this comment was updated there had been 2 new commits pushed to the
Please see this link for an up-to-date comparison between the source branch of this pull request and the ➡️ To integrate this PR with the above commit message to the |
Thanks for the review @jonathan-gibbons ! I'll wait a day in case there are any further comments. |
/integrate |
Going to push as commit 618f3a8.
Your commit was automatically rebased without conflicts. |
@dholmes-ora Pushed as commit 618f3a8. 💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored. |
Mailing list message from David Holmes on javadoc-dev: Hi Patrick, On 6/08/2022 7:05 pm, Patrick Pfeifer wrote:
The sources to the manpages are not currently open sourced and so are
As you found, Oracle does provide these in html format. Cheers, |
Please review these changes to the nroff manpage files so that they match their markdown sources that Oracle maintains.
All pages at a minimum have 19-ea replaced with 19, and copyright set to 2022 if needed. Additionally:
The Java manpage was missing updates from:
The Java manpage has slight formatting differences from:
The Java manpage has a typo fixed in mainline by JDK-8279047 (for JDK 20)
The keytool manpage was missing updates from:
The jar manpage was missing updates from:
The jarsigner manpage was missing updates from:
The javadoc manpage was missing updates from:
--date
optionThe jmod manpage was missing updates from:
The jpackage manpage was missing updates from:
The jshell manpage was missing updates from:
Progress
Issue
Reviewers
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk19 pull/145/head:pull/145
$ git checkout pull/145
Update a local copy of the PR:
$ git checkout pull/145
$ git pull https://git.openjdk.org/jdk19 pull/145/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 145
View PR using the GUI difftool:
$ git pr show -t 145
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk19/pull/145.diff