Skip to content

Conversation

mai93
Copy link
Contributor

@mai93 mai93 commented Jun 20, 2022

Add the JVM options specified via --jvmopt to the jvm_flags of kt_jvm_test targets.

@Bencodes Bencodes merged commit 5abc172 into bazelbuild:master Jun 24, 2022
@Wyverald
Copy link
Member

Wyverald commented Jul 5, 2022

Looks like this change is not compatible with Bazel 4.x: https://buildkite.com/bazel/rules-jvm-external/builds/2741#0181ce76-75c9-4e56-89f6-5af0723beab3

Is there any way we could make this backwards compatible?

copybara-service bot pushed a commit to bazelbuild/intellij that referenced this pull request Jul 12, 2022
rules_kotlin now [support using `--jvmopt` with `kt_jvm_test`](bazelbuild/rules_kotlin#782) .

PiperOrigin-RevId: 460524616
copybara-service bot pushed a commit to bazelbuild/intellij that referenced this pull request Jul 13, 2022
rules_kotlin now [support using `--jvmopt` with `kt_jvm_test`](bazelbuild/rules_kotlin#782) .

PiperOrigin-RevId: 460524616
copybara-service bot pushed a commit to bazelbuild/intellij that referenced this pull request Jul 13, 2022
rules_kotlin now [support using `--jvmopt` with `kt_jvm_test`](bazelbuild/rules_kotlin#782) .

PiperOrigin-RevId: 460731413
mai93 added a commit to bazelbuild/intellij that referenced this pull request Jul 20, 2022
rules_kotlin now [support using `--jvmopt` with `kt_jvm_test`](bazelbuild/rules_kotlin#782) .

(cherry picked from commit 8179705)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants