Switch rules_java back to more named params (quasi-rollback of unknown commit)

PiperOrigin-RevId: 728089692
Change-Id: I76cbe64fbcb422c67db62e87f0bc5eba675191ee

Author: pzembrod

java/common/rules/impl/basic_java_library_impl.bzl

@@ -126,23 +126,23 @@ def basic_java_library(
 
     java_info, compilation_info = compile_action(
         ctx,
-        ctx.outputs.classjar,
-        ctx.outputs.sourcejar,
-        source_files,
-        source_jars,
-        collect_deps(deps) + ([coverage_config.runner] if coverage_config and coverage_config.runner else []),
-        collect_deps(runtime_deps),
-        plugins_javaplugininfo,
-        collect_deps(exports),
-        _collect_plugins(exported_plugins),
-        resources,
-        resource_jars,
-        classpath_resources,
-        _collect_native_libraries(deps, runtime_deps, exports),
-        javacopts,
-        neverlink,
-        ctx.fragments.java.strict_java_deps,
-        enable_compile_jar_action,
+        output_class_jar = ctx.outputs.classjar,
+        output_source_jar = ctx.outputs.sourcejar,
+        source_files = source_files,
+        source_jars = source_jars,
+        deps = collect_deps(deps) + ([coverage_config.runner] if coverage_config and coverage_config.runner else []),
+        runtime_deps = collect_deps(runtime_deps),
+        plugins = plugins_javaplugininfo,
+        exports = collect_deps(exports),
+        exported_plugins = _collect_plugins(exported_plugins),
+        resources = resources,
+        resource_jars = resource_jars,
+        classpath_resources = classpath_resources,
+        native_libraries = _collect_native_libraries(deps, runtime_deps, exports),
+        javacopts = javacopts,
+        neverlink = neverlink,
+        strict_deps = ctx.fragments.java.strict_java_deps,
+        enable_compile_jar_action = enable_compile_jar_action,
         add_exports = add_exports,
         add_opens = add_opens,
         bootclasspath = bootclasspath[BootClassPathInfo] if bootclasspath else None,

java/common/rules/impl/bazel_java_library_impl.bzl

@@ -67,17 +67,17 @@ def bazel_java_library_rule(
 
     target, base_info = basic_java_library(
         ctx,
-        srcs,
-        deps,
-        runtime_deps,
-        plugins,
-        exports,
-        exported_plugins,
-        resources,
-        [],  # resource_jars
-        [],  # class_pathresources
-        javacopts,
-        neverlink,
+        srcs = srcs,
+        deps = deps,
+        runtime_deps = runtime_deps,
+        plugins = plugins,
+        exports = exports,
+        exported_plugins = exported_plugins,
+        resources = resources,
+        resource_jars = [],
+        classpath_resources = [],
+        javacopts = javacopts,
+        neverlink = neverlink,
         proguard_specs = proguard_specs,
         add_exports = add_exports,
         add_opens = add_opens,

java/private/java_common.bzl

@@ -70,8 +70,8 @@ def _compile(
         add_opens = []):
     return _compile_internal(
         ctx,
-        output,
-        java_toolchain,
+        output = output,
+        java_toolchain = java_toolchain,
         source_jars = source_jars,
         source_files = source_files,
         output_source_jar = output_source_jar,