RFR: 8325621: Improve jspawnhelper version checks [v2]

Magnus Ihse Bursie ihse at openjdk.org
Tue Mar 12 14:05:16 UTC 2024


On Mon, 11 Mar 2024 20:04:48 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> 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.

There is already a `$(VERSION_CFLAGS)` variable defined. It will set all those (and some more). Please use it instead. But then, as Roger says, it is probably overkill to *check* anything but the feature version.

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

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


More information about the core-libs-dev mailing list