enable windows/arm64 go build with bazel

[niyas-sait]

What type of PR is this?

Feature

What does this PR do? Why is it needed?

Enable windows/arm64 go build with Bazel

Other notes for review

Go for windows/arm64 is available from version 17. This patch adds the missing pieces to build go programs with bazel.

[niyas-sait]

@linzhp Can you help with review please ?

[linzhp]

Can you try ctx.os.arch?

Suggested change

	goos, goarch = "windows", "amd64"
	if _get_env_var(ctx, "PROCESSOR_ARCHITECTURE") == "ARM64":
	goarch = "arm64"
	elif _get_env_var(ctx, "PROCESSOR_ARCHITEW6432") == "ARM64":
	goarch = "arm64"
	goos = "windows"
	if ctx.os.arch == "<some archecture>":
	goarch = "arm64"
	else:
	goarch = "amd64"

[niyas-sait]

Much better! Seems to work

[niyas-sait]

@linzhp Unfortunately, one test is failing on CI now with the latest change. Any guess?

Error: 'repository_os' value has no field or method 'arch'
  | (18:42:06) ERROR: C:/b/bk-windows-k0sj/bazel/rules-go-golang/WORKSPACE:9:23: fetching _go_download_sdk rule //external:go_sdk: Traceback (most recent call last):
  | File "C:/b/sz2aij3q/external/io_bazel_rules_go/go/private/sdk.bzl", line 47, column 45, in _go_download_sdk_impl
  | goos, goarch = _detect_host_platform(ctx)
  | File "C:/b/sz2aij3q/external/io_bazel_rules_go/go/private/sdk.bzl", line 280, column 18, in _detect_host_platform
  | if ctx.os.arch == "aarch64":
  | Error: 'repository_os' value has no field or method 'arch'
 

[linzhp]

Yeah, repository_os.arch is only available in main branch of Bazel, not yet available in 5.0 release. We can either wait for 5.1 release, or ctx.os.environ.get("PROCESSOR_ARCHITECTURE")

[niyas-sait]

Yeah, repository_os.arch is only available in main branch of Bazel, not yet available in 5.0 release. We can either wait for 5.1 release, or ctx.os.environ.get("PROCESSOR_ARCHITECTURE")

Thanks. Shall I revert latest commit and use previous approach of using environment variables ?

[linzhp]

You can decide which way you want to go

[niyas-sait]

You can decide which way you want to go

Ok I will use env. variable for the time being.

[niyas-sait]

@linzhp Any idea when the next release is expected? I have a related PR for bazelisk that can be applied only after rules_go release

[linzhp]

I am hoping to get #3083 in so next release will be for Go 1.18 support.