Skip to content

Commit

Permalink
Replace ctx.host_configuration.host_path_separator with ctx.configura…
Browse files Browse the repository at this point in the history
…tion.host_path_separator. (#9742)

Replace ctx.host_configuration.host_path_separator with ctx.configuration.host_path_separator.

This is because

1. we're removing ctx.host_configuration
2. host_path_separator's value doesn't depend on the configuration (the API is misleading by embedding it under ctx.configuration). So this is a no-op.

Internal ref cl/494008561.
  • Loading branch information
gregestren authored Dec 14, 2022
1 parent 2a0b86f commit e325dc9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion java_grpc_library.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ def _java_rpc_library_impl(ctx):
args = ctx.actions.args()
args.add(toolchain.plugin, format = "--plugin=protoc-gen-rpc-plugin=%s")
args.add("--rpc-plugin_out={0}:{1}".format(toolchain.plugin_arg, srcjar.path))
args.add_joined("--descriptor_set_in", descriptor_set_in, join_with = ctx.host_configuration.host_path_separator)
args.add_joined("--descriptor_set_in", descriptor_set_in, join_with = ctx.configuration.host_path_separator)
args.add_all(srcs, map_each = _path_ignoring_repository)

ctx.actions.run(
Expand Down

0 comments on commit e325dc9

Please sign in to comment.