hvadehra
hvadehra
Now that bazel 5.0 is released, this is now ready for review @brandjon
Hi @cpsauer, is this something you're still looking at? (btw JavaImport and is now in Starlark: https://github.com/bazelbuild/bazel/issues/15196#issuecomment-1486891278)
We're already doing that by default: https://github.com/bazelbuild/rules_java/blob/master/toolchains/default_java_toolchain.bzl#L47-L50 @ashi009 Are you using a different toolchain configuration?
@cushon What are your thoughts about this?
> Would it make sense to have something similar in rules_java (a separate target), or to add this? That sounds reasonable. @Sineaggi Could you please declare a separate java_runtime with...
Closing in favor of https://github.com/bazelbuild/rules_java/pull/184
> When can we expect a new release of rules_java containing that fix Hopefully soon, I opened https://github.com/bazelbuild/rules_java/issues/186 for now
This is now explained in https://bazel.build/docs/bazel-and-java#java-versions
@josephglanville did https://github.com/bazelbuild/rules_java/issues/102#issuecomment-1676674836 fix this for you?
Alright, feel free to ping here if this comes up again.