hvadehra

Results 96 comments of 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.