RFR: 8325621: Improve jspawnhelper version checks [v2]

Roger Riggs rriggs at openjdk.org
Mon Mar 11 20:07:18 UTC 2024


On Mon, 11 Mar 2024 19:12:33 GMT, Chad Rakoczy <duke at openjdk.org> wrote:

>> Fix for [8325621](https://bugs.openjdk.org/browse/JDK-8325621)
>> 
>> Updates jspawnhelper to check that JDK version and jspawnhelper version are the same. Updates test to include check for version. Also tested manually by replacing jspawnhelper with incorrect version to confirm that check works.
>
> Chad Rakoczy has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Code cleanup

make/modules/java.base/Launcher.gmk line 85:

> 83:           -DVERSION_INTERIM=$(VERSION_INTERIM) \
> 84:           -DVERSION_UPDATE=$(VERSION_UPDATE) \
> 85:           -DVERSION_PATCH=$(VERSION_PATCH), \

Using all 4 is way overkill for the problem at hand.  Just the FEATURE_VERSION is sufficient.
We all know better than to make incompatible changes in minor versions let alone update or patch version.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18204#discussion_r1520364358


More information about the build-dev mailing list