search

bazelbuild / rules_scala

Scala rules for Bazel
Apache License 2.0
363 stars 275 forks source link

Conditionally set -Djava.security.manager=allow for jdk >= 17 #1555

Closed simuons closed 6 months ago

simuons commented 6 months ago

Conditionally set -Djava.security.manager=allow for jdk >= 17

Description

rules_scala uses SecurityManager which is deprecated and fails at runtime on jdk21+ (no replacement for this yet)

Bazel's approach to solve this is conditionally add a jvm flag see https://github.com/bazelbuild/bazel/issues/14502

CI builds were failing for some time because jdk was updated to 21

Motivation

Resolves https://github.com/bazelbuild/rules_scala/issues/1521 Duplicate of https://github.com/bazelbuild/rules_scala/pull/1539