On Wed, 16 Apr 2025 21:51:42 GMT, Kevin Rushforth <[email protected]> wrote:
> > yes, we still need `SWT_TEST` so we can disable it on mac.
>
> If that flag is the mechanism we need to use to disable it on macOS, then we
> will need the following additional change:
>
> ```
> // Specifies whether to run system tests that depend on SWT (only used when
> FULL_TEST is also enabled)
> -defineProperty("SWT_TEST", "true")
> +defineProperty("SWT_TEST", IS_MAC ? "false" ? "true")
> ext.IS_SWT_TEST = Boolean.parseBoolean(SWT_TEST);
> ```
Never mind. There is already this:
if (IS_MAC) {
enabled = false
logger.info("SWT tests are disabled on MAC, because Gradle test
runner does not handle -XstartOnFirstThread properly
(https://issues.gradle.org/browse/GRADLE-3290).")
}
-------------
PR Comment: https://git.openjdk.org/jfx/pull/1783#issuecomment-2810909886
