Try fixing windows

go/private/rules/BUILD.bazel

@@ -28,6 +28,7 @@ bzl_library(
         "//go/private:mode",
         "//go/private:providers",
         "//go/private/rules:transition",
+        "//go/private/skylib/lib:copy_file",
     ],
 )
 
@@ -117,6 +118,7 @@ bzl_library(
         "//go/private:providers",
         "//go/private/rules:binary",
         "//go/private/rules:transition",
+        "//go/private/skylib/lib:copy_file",
         "@bazel_skylib//lib:structs",
     ],
 )

go/private/rules/binary.bzl

@@ -33,6 +33,11 @@ load(
     "go_transition",
     "transition_attrs",
 )
+load(
+    "//go/private/skylib/lib:copy_file.bzl",
+    "copy_bash",
+    "copy_cmd",
+)
 load(
     "//go/private:mode.bzl",
     "LINKMODE_C_ARCHIVE",
@@ -393,11 +398,10 @@ def _forward_binary_output(ctx):
     di = ctx.attr.transition_dep[0][DefaultInfo]
 
     copied_executable = ctx.actions.declare_file(ctx.label.name)
-    ctx.actions.symlink(
-        output = copied_executable,
-        target_file = ep.executable,
-        is_executable = True,
-    )
+    if ctx.attr.is_windows:
+        copy_cmd(ctx, ep.executable, copied_executable)
+    else:
+        copy_bash(ctx, ep.executable, copied_executable)
 
     data_runfiles = di.data_runfiles.merge(ctx.runfiles([copied_executable]))
     default_runfiles = di.default_runfiles.merge(ctx.runfiles([copied_executable]))
@@ -411,7 +415,10 @@ def _forward_binary_output(ctx):
 
 go_transition_binary = rule(
     implementation = _forward_binary_output,
-    attrs = _merge(transition_attrs, {"transition_dep": attr.label(cfg = go_transition)}),
+    attrs = _merge(transition_attrs, {
+        "is_windows": attr.bool(),
+        "transition_dep": attr.label(cfg = go_transition),
+    }),
     executable = True,
 )
 

go/private/rules/test.bzl

@@ -43,6 +43,11 @@ load(
     "go_transition",
     "transition_attrs",
 )
+load(
+    "//go/private/skylib/lib:copy_file.bzl",
+    "copy_bash",
+    "copy_cmd",
+)
 load(
     "//go/private:mode.bzl",
     "LINKMODE_NORMAL",
@@ -65,11 +70,11 @@ def _forward_test_output(ctx):
     di = ctx.attr.transition_dep[0][DefaultInfo]
 
     copied_executable = ctx.actions.declare_file(ctx.label.name)
-    ctx.actions.symlink(
-        output = copied_executable,
-        target_file = ep.executable,
-        is_executable = True,
-    )
+
+    if ctx.attr.is_windows:
+        copy_cmd(ctx, ep.executable, copied_executable)
+    else:
+        copy_bash(ctx, ep.executable, copied_executable)
 
     data_runfiles = di.data_runfiles.merge(ctx.runfiles([copied_executable]))
     default_runfiles = di.default_runfiles.merge(ctx.runfiles([copied_executable]))
@@ -85,6 +90,7 @@ go_transition_test = rule(
     implementation = _forward_test_output,
     attrs = _merge(transition_attrs, {
         "transition_dep": attr.label(cfg = go_transition),
+        "is_windows": attr.bool(),
         # Workaround for bazelbuild/bazel#6293. See comment in lcov_merger.sh.
         "_lcov_merger": attr.label(
             executable = True,

go/private/rules/transition.bzl

@@ -95,7 +95,14 @@ def go_transition_wrapper(kind, transition_kind, name, use_basename, keys_to_str
         # but preserve things like tags and exec_compatible_with,
         # which are general to all tests rather than specific to go_test specifically,
         # and should apply to generated tests and binaries too.
-        transition_kind(name = name, transition_dep = transitioned_name, **_minus_keys(kwargs, keys_to_strip))
+        transition_kwargs = _minus_keys(kwargs, keys_to_strip)
+        transition_kwargs["name"] = name
+        transition_kwargs["transition_dep"] = transitioned_name
+        transition_kwargs["is_windows"] = select({
+            "@bazel_tools//src/conditions:windows": True,
+            "//conditions:default": False,
+        })
+        transition_kind(**transition_kwargs)
         tags = kwargs.pop("tags", [])
         if "manual" not in tags:
             tags += ["manual"]

go/private/skylib/lib/BUILD.bazel

@@ -18,3 +18,9 @@ bzl_library(
     srcs = ["versions.bzl"],
     visibility = ["//go:__subpackages__"],
 )
+
+bzl_library(
+    name = "copy_file",
+    srcs = ["copy_file.bzl"],
+    visibility = ["//go:__subpackages__"],
+)