RFR: 8255713: JavaFX build should discover Visual Studio compiler on system

Kevin Rushforth kcr at openjdk.java.net
Tue Mar 16 16:35:19 UTC 2021


We currently hard-code the default version of Visual Studio in both `win.gradle` and `.github/workflows/submit.yml`. This hard-coding of the specific version of MSVC is fragile, particularly for GitHub action builds. The last time Microsoft changed the environment, our Windows builds started failing (because we hard-coded it). I temporarily bumped it to the then-current version as a fix for [JDK-8256686](https://bugs.openjdk.java.net/browse/JDK-8256686), and filed this bug -- [JDK-8255713](https://bugs.openjdk.java.net/browse/JDK-8255713) -- as a followup. It's failing again because of another update to the GitHub actions environment, so it's time to fix it to discover the latest version rather than hardcode it.

NOTE to reviewers: This will need to be tested with local and CI builds as well as GitHub actions, since there are changes in the `win.gradle` script (and associated `genVSproperties.bat` script).

-------------

Commit messages:
 - 8255713: JavaFX build should discover Visual Studio compiler on system

Changes: https://git.openjdk.java.net/jfx/pull/427/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jfx&pr=427&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8255713
  Stats: 25 lines in 3 files changed: 17 ins; 2 del; 6 mod
  Patch: https://git.openjdk.java.net/jfx/pull/427.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/427/head:pull/427

PR: https://git.openjdk.java.net/jfx/pull/427


More information about the openjfx-dev mailing list