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 build-dev
mailing list