Re: RFR: 8257722: Improve "keytool -printcert -jarfile" output [v2]

2021-10-27 Thread Weijun Wang
On Wed, 27 Oct 2021 16:32:48 GMT, Hai-May Chao  wrote:

>> This change does a few improvements to the output of `keytool -printcert 
>> -jarfile` command to help readability and diagnosis.
>
> Hai-May Chao has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Updated TimestampCheck test and removed its unused methods

One new comment: Now that if there is only one cert, there is no need to write 
`Certificate #n of m`. If there's only one signer, we also don't need to write 
`of signer #n`.

-

PR: https://git.openjdk.java.net/jdk/pull/6126


Re: RFR: 8257722: Improve "keytool -printcert -jarfile" output [v2]

2021-10-27 Thread Hai-May Chao
> This change does a few improvements to the output of `keytool -printcert 
> -jarfile` command to help readability and diagnosis.

Hai-May Chao has updated the pull request incrementally with one additional 
commit since the last revision:

  Updated TimestampCheck test and removed its unused methods

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/6126/files
  - new: https://git.openjdk.java.net/jdk/pull/6126/files/ba4bb4b7..02f723d5

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=6126=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=6126=00-01

  Stats: 46 lines in 1 file changed: 0 ins; 45 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/6126.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/6126/head:pull/6126

PR: https://git.openjdk.java.net/jdk/pull/6126