Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v6]

2024-11-19 Thread Johan Vos
On Tue, 19 Nov 2024 04:34:05 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-19 Thread Kevin Rushforth
On Sun, 17 Nov 2024 09:02:56 GMT, Johan Vos wrote: >> If we do this, then here are some possible names: >> >> * jfx.experimental.release >> * jfx.experimental.release.name >> * jfx.variant.release >> * jfx.variant.release.name >> * jfx.variant >> * jfx.feature.variant >> >> Thoughts? > >> If we

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v6]

2024-11-19 Thread Kevin Rushforth
On Tue, 19 Nov 2024 04:34:05 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v5]

2024-11-18 Thread Ambarish Rapte
On Mon, 18 Nov 2024 21:13:11 GMT, Kevin Rushforth 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:

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v6]

2024-11-18 Thread Ambarish Rapte
> Introduce a new build property `jfx.experimental.release.suffix` to be used > for the early access builds of an under development feature in JavaFX. > This property would be set to a value ONLY in a branch specific to an > experiment in [jfx-sandbox](https://github.com/openjdk/jfx-sandbox) repo

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-18 Thread Nir Lisker
On Fri, 15 Nov 2024 05:48:15 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v5]

2024-11-18 Thread Kevin Rushforth
On Mon, 18 Nov 2024 20:56:44 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v5]

2024-11-18 Thread Ambarish Rapte
> Introduce a new build property `jfx.experimental.release.suffix` to be used > for the early access builds of an under development feature in JavaFX. > This property would be set to a value ONLY in a branch specific to an > experiment in [jfx-sandbox](https://github.com/openjdk/jfx-sandbox) repo

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v3]

2024-11-18 Thread Ambarish Rapte
On Mon, 18 Nov 2024 14:46:57 GMT, Kevin Rushforth wrote: > `jfx.experimental.feature.name` -- The name of the experimental feature Thanks @kevinrushforth , Updated the property name as suggested. - PR Comment: https://git.openjdk.org/jfx/pull/1637#issuecomment-2483775795

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v4]

2024-11-18 Thread Ambarish Rapte
> Introduce a new build property `jfx.experimental.release.suffix` to be used > for the early access builds of an under development feature in JavaFX. > This property would be set to a value ONLY in a branch specific to an > experiment in [jfx-sandbox](https://github.com/openjdk/jfx-sandbox) repo

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v3]

2024-11-18 Thread Kevin Rushforth
On Mon, 18 Nov 2024 11:52:12 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v3]

2024-11-18 Thread Ambarish Rapte
On Mon, 18 Nov 2024 11:52:12 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v3]

2024-11-18 Thread Ambarish Rapte
> Introduce a new build property `jfx.experimental.release.suffix` to be used > for the early access builds of an under development feature in JavaFX. > This property would be set to a value ONLY in a branch specific to an > experiment in [jfx-sandbox](https://github.com/openjdk/jfx-sandbox) repo

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-18 Thread Ambarish Rapte
On Sat, 16 Nov 2024 15:12:29 GMT, Kevin Rushforth wrote: >> Ambarish Rapte has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains two additional >> com

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-18 Thread Ambarish Rapte
On Sun, 17 Nov 2024 09:06:00 GMT, Johan Vos wrote: >>> I know I'm a broken record on this topic, but if possible, it would be >>> great if (parts of) this logic can be moved into a separate file in order >>> to make the build.gradle smaller and more readable. >> >> Also at the risk of sounding

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-17 Thread Johan Vos
On Sat, 16 Nov 2024 22:36:17 GMT, Kevin Rushforth wrote: > If we do this, then here are some possible names: > > * jfx.experimental.release > * jfx.experimental.release.name > * jfx.variant.release > * jfx.variant.release.name > * jfx.variant > * jfx.feature.variant > > Thoughts? I don't have

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-17 Thread Johan Vos
On Sat, 16 Nov 2024 22:32:30 GMT, Kevin Rushforth wrote: > > It is a bit confusing to have "jfx.release.suffix" and > > "jfx.experimental.release.suffix" where the former refers to an artifact > > property and the latter refers to a major feature/variant/... like metal. > > I think you have a

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-17 Thread Johan Vos
On Sat, 16 Nov 2024 19:43:07 GMT, Nir Lisker wrote: >> I agree with that. It is often useful to have timestamp info in the builds >> during development. >> I know I'm a broken record on this topic, but if possible, it would be great >> if (parts of) this logic can be moved into a separate file

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-16 Thread Kevin Rushforth
On Sat, 16 Nov 2024 18:32:10 GMT, Johan Vos wrote: > It is a bit confusing to have "jfx.release.suffix" and > "jfx.experimental.release.suffix" where the former refers to an artifact > property and the latter refers to a major feature/variant/... like metal. I think you have a good point. May

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-16 Thread Kevin Rushforth
On Fri, 15 Nov 2024 05:48:15 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-16 Thread Nir Lisker
On Sat, 16 Nov 2024 18:27:02 GMT, Johan Vos wrote: > I know I'm a broken record on this topic, but if possible, it would be great > if (parts of) this logic can be moved into a separate file in order to make > the build.gradle smaller and more readable. Also at the risk of sounding like a brok

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-16 Thread Johan Vos
On Sat, 16 Nov 2024 16:16:58 GMT, Kevin Rushforth wrote: > > I might be confused, but wasn't the idea to use a new property (e.g. > > jfx.feature.variant) instead of overloading the existing jfx.release.suffix? > > This PR does add a new property in > `build.properties`,`jfx.experimental.relea

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-16 Thread Kevin Rushforth
On Fri, 15 Nov 2024 05:48:15 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-16 Thread Johan Vos
On Sat, 16 Nov 2024 15:09:40 GMT, Kevin Rushforth wrote: >> Ambarish Rapte has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains two additional >> com

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-16 Thread Kevin Rushforth
On Sat, 16 Nov 2024 15:37:22 GMT, Johan Vos wrote: > I might be confused, but wasn't the idea to use a new property (e.g. > jfx.feature.variant) instead of overloading the existing jfx.release.suffix? This PR does add a new property in `build.properties`,`jfx.experimental.release.suffix`, and

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-16 Thread Johan Vos
On Fri, 15 Nov 2024 05:48:15 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX

2024-11-15 Thread Ambarish Rapte
On Thu, 14 Nov 2024 12:02:02 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX [v2]

2024-11-14 Thread Ambarish Rapte
> Introduce a new build property `jfx.experimental.release.suffix` to be used > for the early access builds of an under development feature in JavaFX. > This property would be set to a value ONLY in a branch specific to an > experiment in [jfx-sandbox](https://github.com/openjdk/jfx-sandbox) repo

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX

2024-11-14 Thread Kevin Rushforth
On Thu, 14 Nov 2024 12:02:02 GMT, Ambarish Rapte wrote: >> Introduce a new build property `jfx.experimental.release.suffix` to be used >> for the early access builds of an under development feature in JavaFX. >> This property would be set to a value ONLY in a branch specific to an >> experiment

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX

2024-11-14 Thread Kevin Rushforth
On Thu, 14 Nov 2024 12:01:29 GMT, Ambarish Rapte wrote: > Introduce a new build property `jfx.experimental.release.suffix` to be used > for the early access builds of an under development feature in JavaFX. > This property would be set to a value ONLY in a branch specific to an > experiment in

RFR: 8343196: Add build property to identify experimental builds of JavaFX

2024-11-14 Thread Ambarish Rapte
Introduce a new build property `jfx.experimental.release.suffix` to be used for the early access builds of an under development feature in JavaFX. This property would be set to a value ONLY in a branch specific to an experiment in [jfx-sandbox](https://github.com/openjdk/jfx-sandbox) repo. For ex

Re: RFR: 8343196: Add build property to identify experimental builds of JavaFX

2024-11-14 Thread Ambarish Rapte
On Thu, 14 Nov 2024 12:01:29 GMT, Ambarish Rapte wrote: > Introduce a new build property `jfx.experimental.release.suffix` to be used > for the early access builds of an under development feature in JavaFX. > This property would be set to a value ONLY in a branch specific to an > experiment in