Re: RFR: 8309381: Support JavaFX incubator modules [v3]

2024-12-02 Thread Andy Goryachev
On Mon, 2 Dec 2024 17:04:06 GMT, Kevin Rushforth wrote: >> build.gradle line 2877: >> >>> 2875: // Add a project declaration for each incubator module here, leaving >>> the >>> 2876: // incubator placeholder lines as an example. >>> 2877: // BEGIN: incubator placeholder >> >> Q: maybe we shoul

Re: RFR: 8309381: Support JavaFX incubator modules [v3]

2024-12-02 Thread Kevin Rushforth
> This PR add the necessary support for [JavaFX Incubator > Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). > It includes the following: > > 1. Changes to the build scripts `build.gradle` and `settings.gradle` to > document where to add your incubator mo

Re: RFR: 8309381: Support JavaFX incubator modules [v3]

2024-12-02 Thread Andy Goryachev
On Mon, 2 Dec 2024 17:22:07 GMT, Kevin Rushforth wrote: >> This PR add the necessary support for [JavaFX Incubator >> Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). >> It includes the following: >> >> 1. Changes to the build scripts `build.gradle` and

Re: RFR: 8309381: Support JavaFX incubator modules [v3]

2024-12-02 Thread Kevin Rushforth
On Mon, 2 Dec 2024 17:22:07 GMT, Kevin Rushforth wrote: >> This PR add the necessary support for [JavaFX Incubator >> Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). >> It includes the following: >> >> 1. Changes to the build scripts `build.gradle` and

Re: RFR: 8309381: Support JavaFX incubator modules [v3]

2024-12-02 Thread Andy Goryachev
On Mon, 2 Dec 2024 17:22:07 GMT, Kevin Rushforth wrote: >> This PR add the necessary support for [JavaFX Incubator >> Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). >> It includes the following: >> >> 1. Changes to the build scripts `build.gradle` and

Re: RFR: 8309381: Support JavaFX incubator modules [v3]

2024-12-02 Thread Kevin Rushforth
On Tue, 5 Nov 2024 22:31:14 GMT, Andy Goryachev wrote: >> Kevin Rushforth 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 six additional >> comm

Re: RFR: 8309381: Support JavaFX incubator modules [v2]

2024-11-27 Thread Nir Lisker
On Fri, 22 Nov 2024 16:36:02 GMT, Kevin Rushforth wrote: >> This PR add the necessary support for [JavaFX Incubator >> Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). >> It includes the following: >> >> 1. Changes to the build scripts `build.gradle` and

Re: RFR: 8309381: Support JavaFX incubator modules [v2]

2024-11-23 Thread Kevin Rushforth
On Sat, 23 Nov 2024 02:10:21 GMT, Nir Lisker wrote: >> Kevin Rushforth 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 three additional >> commi

Re: RFR: 8309381: Support JavaFX incubator modules [v2]

2024-11-23 Thread Johan Vos
On Sat, 23 Nov 2024 02:10:21 GMT, Nir Lisker wrote: > I had a look (at v2). If this approach works then it's fine for now, but it > repeats the sin of coupling configurations of (potentially) unrelated modules > and will increase the build time for a normal build (at least it will from > the l

Re: RFR: 8309381: Support JavaFX incubator modules [v2]

2024-11-22 Thread Nir Lisker
On Fri, 22 Nov 2024 16:36:02 GMT, Kevin Rushforth wrote: >> This PR add the necessary support for [JavaFX Incubator >> Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). >> It includes the following: >> >> 1. Changes to the build scripts `build.gradle` and

Re: RFR: 8309381: Support JavaFX incubator modules [v2]

2024-11-22 Thread Kevin Rushforth
On Fri, 22 Nov 2024 19:57:15 GMT, Andy Goryachev wrote: > could you create a draft PR for v2, so we can leave comments? Done. I created Draft PR #1646 - PR Comment: https://git.openjdk.org/jfx/pull/1616#issuecomment-2494748813

Re: RFR: 8309381: Support JavaFX incubator modules [v2]

2024-11-22 Thread Andy Goryachev
On Tue, 5 Nov 2024 22:38:52 GMT, Andy Goryachev wrote: >> Kevin Rushforth 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 three additional >> co

Re: RFR: 8309381: Support JavaFX incubator modules [v2]

2024-11-22 Thread Kevin Rushforth
On Fri, 22 Nov 2024 16:36:02 GMT, Kevin Rushforth wrote: >> This PR add the necessary support for [JavaFX Incubator >> Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). >> It includes the following: >> >> 1. Changes to the build scripts `build.gradle` and

Re: RFR: 8309381: Support JavaFX incubator modules [v2]

2024-11-22 Thread Kevin Rushforth
> This PR add the necessary support for [JavaFX Incubator > Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). > It includes the following: > > 1. Changes to the build scripts `build.gradle` and `settings.gradle` to > document where to add your incubator mo

Re: RFR: 8309381: Support JavaFX incubator modules

2024-11-14 Thread Kevin Rushforth
On Thu, 14 Nov 2024 09:32:33 GMT, Johan Vos wrote: >> This PR add the necessary support for [JavaFX Incubator >> Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). >> It includes the following: >> >> 1. Changes to the build scripts `build.gradle` and `sett

Re: RFR: 8309381: Support JavaFX incubator modules

2024-11-14 Thread Johan Vos
On Tue, 29 Oct 2024 22:23:35 GMT, Kevin Rushforth wrote: > This PR add the necessary support for [JavaFX Incubator > Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). > It includes the following: > > 1. Changes to the build scripts `build.gradle` and `set

Re: RFR: 8309381: Support JavaFX incubator modules

2024-11-14 Thread Ambarish Rapte
On Tue, 29 Oct 2024 22:23:35 GMT, Kevin Rushforth wrote: > This PR add the necessary support for [JavaFX Incubator > Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). > It includes the following: > > 1. Changes to the build scripts `build.gradle` and `set

Re: RFR: 8309381: Support JavaFX incubator modules

2024-11-05 Thread Andy Goryachev
On Tue, 29 Oct 2024 22:23:35 GMT, Kevin Rushforth wrote: > This PR add the necessary support for [JavaFX Incubator > Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). > It includes the following: > > 1. Changes to the build scripts `build.gradle` and `set

Re: RFR: 8309381: Support JavaFX incubator modules

2024-11-01 Thread Kevin Rushforth
On Tue, 29 Oct 2024 22:23:35 GMT, Kevin Rushforth wrote: > This PR add the necessary support for [JavaFX Incubator > Modules](https://github.com/kevinrushforth/jfx/blob/jfx.incubator/INCUBATOR-MODULES.md). > It includes the following: > > 1. Changes to the build scripts `build.gradle` and `set