> Hi all, > > Please review this fix for jar tool not producing an archive if there is a > missing file supplied. > The current behaviour will recognise missing files as an error but continue > processing, > creating a temporary archive and then deleting it without moving to the > current directory. > The fix is to return false when a missing file is supplied and exit > immediately without continuing with any wasted processing. > > Thanks, > Ryan.
Ryan Wallace 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 eight additional commits since the last revision: - 8318971: Better Error Handling for Jar Tool Processing of "@File" - Merge branch 'master' into 8318971 - Merge branch 'master' into 8318971 - 8318971: jar v17 should either exit on error immediately or create archive as jar v1.8 did - 8318971: jar v17 should either exit on error immediately or create archive as jar v1.8 did - Merge branch 'master' into 8318971 - 8318971: jar v17 should either exit on error immediately or create archive as jar v1.8 did - 8318971: jar v17 should either exit on error immediately or create archive as jar v1.8 did ------------- Changes: - all: https://git.openjdk.org/jdk/pull/16423/files - new: https://git.openjdk.org/jdk/pull/16423/files/c7b18741..cee20934 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=16423&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16423&range=02-03 Stats: 632825 lines in 1230 files changed: 90078 ins; 478477 del; 64270 mod Patch: https://git.openjdk.org/jdk/pull/16423.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/16423/head:pull/16423 PR: https://git.openjdk.org/jdk/pull/16423