RFR: 8325342: Remove unneeded exceptions in compare.sh

Erik Joelsson erikj at openjdk.org
Tue Feb 6 17:57:55 UTC 2024


On Tue, 6 Feb 2024 16:53:37 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

> Over time, we have been better at addressing inconsistencies in the build, but the exceptions put in place in compare.sh have not been updated to reflect this.
> 
> This attempts to make sure we only keep those exceptions that are currently actually needed to verify a reproducible build (the primary usecase for the compare script these days).
> 
> I also discovered bugs in the script, where mismatches in a zip file did not make us report a failed comparison. I added extra code to make sure we always set the REGRESSION flag whenever we detect a problem.

make/scripts/compare.sh line 298:

> 296:         | $GREP -v "/hotspot/gtest/server/gtestLauncher" \
> 297:         | $GREP -v "/hotspot/gtest/server/libjvm.dylib" \
> 298:         | $SORT | $FILTER)

This list of exclusion, or at least most of it, isn't for handling exceptions but because those files are handled explicitly by other comparison functions.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/17735#discussion_r1480337401


More information about the build-dev mailing list