Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v4]

2024-08-14 Thread Marius Hanl
On Wed, 14 Aug 2024 02:21:30 GMT, Kevin Rushforth wrote: >> 1. Added logic to look in the standard locations for Visual Studio 2022, >> with a fallback to 2019 and then 2017 if 2022 is not found. It will look in >> the following locations: >> >> C:("Program Files"|"Program Files (x86)")\Micros

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v4]

2024-08-14 Thread Nir Lisker
On Wed, 14 Aug 2024 02:21:30 GMT, Kevin Rushforth wrote: >> 1. Added logic to look in the standard locations for Visual Studio 2022, >> with a fallback to 2019 and then 2017 if 2022 is not found. It will look in >> the following locations: >> >> C:("Program Files"|"Program Files (x86)")\Micros

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v4]

2024-08-14 Thread Lukasz Kostyra
On Wed, 14 Aug 2024 02:21:30 GMT, Kevin Rushforth wrote: >> 1. Added logic to look in the standard locations for Visual Studio 2022, >> with a fallback to 2019 and then 2017 if 2022 is not found. It will look in >> the following locations: >> >> C:("Program Files"|"Program Files (x86)")\Micros

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v4]

2024-08-13 Thread Kevin Rushforth
> 1. Added logic to look in the standard locations for Visual Studio 2022, with > a fallback to 2019 and then 2017 if 2022 is not found. It will look in the > following locations: > > C:("Program Files"|"Program Files (x86)")\Microsoft Visual > Studio(2022|2019|2017)\ > (Enterprise|Profess

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-13 Thread Kevin Rushforth
On Tue, 13 Aug 2024 06:27:51 GMT, Lukasz Kostyra wrote: >> Thanks, that would be helpful. Let me know what you find. > > I checked on a Win 11 system where Build Tools 2022 are installed in default > directory (no other Visual Studio edition) and the build failed with "Cannot > locate Visual St

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v3]

2024-08-13 Thread Marius Hanl
On Mon, 12 Aug 2024 21:31:49 GMT, Kevin Rushforth wrote: >> 1. Added logic to look in the standard locations for Visual Studio 2022, >> with a fallback to 2019 and then 2017 if 2022 is not found. It will look in >> the following locations: >> >> C:("Program Files"|"Program Files (x86)")\Micros

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v3]

2024-08-13 Thread Nir Lisker
On Mon, 12 Aug 2024 21:31:49 GMT, Kevin Rushforth wrote: >> 1. Added logic to look in the standard locations for Visual Studio 2022, >> with a fallback to 2019 and then 2017 if 2022 is not found. It will look in >> the following locations: >> >> C:("Program Files"|"Program Files (x86)")\Micros

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-12 Thread Lukasz Kostyra
On Mon, 12 Aug 2024 18:04:47 GMT, Kevin Rushforth wrote: >> buildSrc/genVSproperties.bat line 52: >> >>> 50: for %%a in (2022, 2019, 2017) do ( >>> 51: set year=%%a >>> 52: for %%b in (Enterprise, Professional, Community) do ( >> >> Quick two cents from me on this part - Mic

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v3]

2024-08-12 Thread Phil Race
On Mon, 12 Aug 2024 21:31:49 GMT, Kevin Rushforth wrote: >> 1. Added logic to look in the standard locations for Visual Studio 2022, >> with a fallback to 2019 and then 2017 if 2022 is not found. It will look in >> the following locations: >> >> C:("Program Files"|"Program Files (x86)")\Micros

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-12 Thread Kevin Rushforth
On Mon, 12 Aug 2024 20:31:37 GMT, Phil Race wrote: >> Kevin Rushforth has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Revert "Add debug prints" >> >> This reverts commit 3ea8ee58867d21e5c0aeb6a22170cdc28dd7a486. > > buildSrc/genVSpr

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v3]

2024-08-12 Thread Kevin Rushforth
On Mon, 12 Aug 2024 20:41:13 GMT, Phil Race wrote: >> Kevin Rushforth has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Review comments > > buildSrc/win.gradle line 93: > >> 91: defineProperty("WINDOWS_VS_LIB", properties, >> "$W

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v3]

2024-08-12 Thread Kevin Rushforth
> 1. Added logic to look in the standard locations for Visual Studio 2022, with > a fallback to 2019 and then 2017 if 2022 is not found. It will look in the > following locations: > > C:("Program Files"|"Program Files (x86)")\Microsoft Visual > Studio(2022|2019|2017)\ > (Enterprise|Profess

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-12 Thread Phil Race
On Mon, 12 Aug 2024 15:37:10 GMT, Kevin Rushforth wrote: >> 1. Added logic to look in the standard locations for Visual Studio 2022, >> with a fallback to 2019 and then 2017 if 2022 is not found. It will look in >> the following locations: >> >> C:("Program Files"|"Program Files (x86)")\Micros

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-12 Thread Phil Race
On Mon, 12 Aug 2024 19:48:11 GMT, Kevin Rushforth wrote: > > The wiki page for building OpenJFX will also need an update. > > Yes. I plan to (I meant to list that in the Description, but forgot to add > it). Thanks for the reminder. Yes indeed, Kevin and I discussed that at some length. What i

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-12 Thread Kevin Rushforth
On Mon, 12 Aug 2024 19:18:40 GMT, Nir Lisker wrote: > The wiki page for building OpenJFX will also need an update. Yes. I plan to (I meant to list that in the Description, but forgot to add it). Thanks for the reminder. - PR Comment: https://git.openjdk.org/jfx/pull/1534#issuecomm

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-12 Thread Nir Lisker
On Mon, 12 Aug 2024 15:37:10 GMT, Kevin Rushforth wrote: >> 1. Added logic to look in the standard locations for Visual Studio 2022, >> with a fallback to 2019 and then 2017 if 2022 is not found. It will look in >> the following locations: >> >> C:("Program Files"|"Program Files (x86)")\Micros

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-12 Thread Kevin Rushforth
On Mon, 12 Aug 2024 17:32:10 GMT, Lukasz Kostyra wrote: >> Kevin Rushforth has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Revert "Add debug prints" >> >> This reverts commit 3ea8ee58867d21e5c0aeb6a22170cdc28dd7a486. > > buildSrc/ge

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-12 Thread Lukasz Kostyra
On Mon, 12 Aug 2024 15:37:10 GMT, Kevin Rushforth wrote: >> 1. Added logic to look in the standard locations for Visual Studio 2022, >> with a fallback to 2019 and then 2017 if 2022 is not found. It will look in >> the following locations: >> >> C:("Program Files"|"Program Files (x86)")\Micros

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-12 Thread Kevin Rushforth
On Mon, 12 Aug 2024 15:37:10 GMT, Kevin Rushforth wrote: >> 1. Added logic to look in the standard locations for Visual Studio 2022, >> with a fallback to 2019 and then 2017 if 2022 is not found. It will look in >> the following locations: >> >> C:("Program Files"|"Program Files (x86)")\Micros

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set [v2]

2024-08-12 Thread Kevin Rushforth
> 1. Added logic to look in the standard locations for Visual Studio 2022, with > a fallback to 2019 and then 2017 if 2022 is not found. It will look in the > following locations: > > C:("Program Files"|"Program Files (x86)")\Microsoft Visual > Studio(2022|2019|2017)\ > (Enterprise|Profess

Re: RFR: 8289174: JavaFX build fails on Windows when VS150COMNTOOLS is not set

2024-08-12 Thread Kevin Rushforth
On Mon, 12 Aug 2024 14:21:14 GMT, Kevin Rushforth wrote: > 1. Added logic to look in the standard locations for Visual Studio 2022, with > a fallback to 2019 and then 2017 if 2022 is not found. It will look in the > following locations: > > C:("Program Files"|"Program Files (x86)")\Microsoft V