RFR: 8343196: Add build property to identify experimental builds of JavaFX [v5]
Ambarish Rapte
arapte at openjdk.org
Tue Nov 19 04:36:48 UTC 2024
On Mon, 18 Nov 2024 21:13:11 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> Ambarish Rapte has updated the pull request incrementally with one additional commit since the last revision:
>>
>> correcttion
>
> build.gradle line 706:
>
>> 704: relSuffix = jfxExperimentalFeatureName != "" ?
>> 705: "-${jfxExperimentalFeatureName}" : jfxReleaseSuffix;
>> 706: relOpt = "-${buildTimestamp}"
>
> This will unconditionally add a timestamp to _all_ EA builds of JavaFX, not just experimental builds. Changing the version string for EA builds seems out of scope for this PR.
>
> Suggestion:
>
> if (jfxExperimentalFeatureName != "") {
> relSuffix = "-${jfxExperimentalFeatureName}"
> relOpt = "-${buildTimestamp}"
> } else {
> relSuffix = jfxReleaseSuffix
> }
Thanks @kevinrushforth , Included the change.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1637#discussion_r1847626483
More information about the openjfx-dev
mailing list