RFR: 8341775: Duplicate manifest files are removed by jarsigner after signing [v6]
Kevin Driver
kdriver at openjdk.org
Thu Mar 6 21:43:59 UTC 2025
> JDK-8341775: In the case where there is a *single* META-INF directory but potentially *multiple* manifest files of different cases, print a warning before selecting the first one and ignoring the rest (the current behavior should be maintained).
>
> **Note**: We cannot (so far) pass whether the verbose flag is set to the class that does this processing. We may want to add a property to the builder for this. As-is, the message will be printed via `System.err` whether verbose is set or not.
Kevin Driver has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
- Merge remote-tracking branch 'origin/master' into 8341775
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
- Merge branch 'master' of github.com:openjdk/jdk into 8341775
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
- Merge branch '8341775' of gh:driverkt/jdk into 8341775
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
- Merge branch 'master' of https://github.com/openjdk/jdk into 8341775
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
- Merge branch 'master' of github.com:openjdk/jdk into 8341775
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
- Merge branch 'master' of github.com:openjdk/jdk into 8341775
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
- JDK-8341775: In the case where there is a *single* META-INF directory but potentially *multiple* manifest files of different cases, print a warning before selecting the first one and ignoring the rest.
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/22222/files
- new: https://git.openjdk.org/jdk/pull/22222/files/e39390aa..2b133a31
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=22222&range=05
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=22222&range=04-05
Stats: 229902 lines in 5423 files changed: 114349 ins; 90681 del; 24872 mod
Patch: https://git.openjdk.org/jdk/pull/22222.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/22222/head:pull/22222
PR: https://git.openjdk.org/jdk/pull/22222
More information about the security-dev
mailing list