diff options
Diffstat (limited to 'go/private/rules/binary.bzl')
| -rw-r--r-- | go/private/rules/binary.bzl | 41 |
1 files changed, 26 insertions, 15 deletions
diff --git a/go/private/rules/binary.bzl b/go/private/rules/binary.bzl index ff57f4ec..91748eda 100644 --- a/go/private/rules/binary.bzl +++ b/go/private/rules/binary.bzl @@ -122,13 +122,28 @@ def _go_binary_impl(ctx): executable = executable, ) + providers = [ + library, + source, + archive, + OutputGroupInfo( + cgo_exports = archive.cgo_exports, + compilation_outputs = [archive.data.file], + ), + ] + if go.mode.link in LINKMODES_EXECUTABLE: + env = {} + for k, v in ctx.attr.env.items(): + env[k] = ctx.expand_location(v, ctx.attr.data) + providers.append(RunEnvironmentInfo(environment = env)) + # The executable is automatically added to the runfiles. - default_info = DefaultInfo( + providers.append(DefaultInfo( files = depset([executable]), runfiles = runfiles, executable = executable, - ) + )) else: # Workaround for https://github.com/bazelbuild/bazel/issues/15043 # As of Bazel 5.1.1, native rules do not pick up the "files" of a data @@ -136,22 +151,11 @@ def _go_binary_impl(ctx): # non-data dependents should not pick up the executable as a runfile # implicitly, the deprecated "default_runfiles" and "data_runfiles" # constructor parameters have to be used. - default_info = DefaultInfo( + providers.append(DefaultInfo( files = depset([executable]), default_runfiles = runfiles, data_runfiles = runfiles.merge(ctx.runfiles([executable])), - ) - - providers = [ - library, - source, - archive, - default_info, - OutputGroupInfo( - cgo_exports = archive.cgo_exports, - compilation_outputs = [archive.data.file], - ), - ] + )) # If the binary's linkmode is c-archive or c-shared, expose CcInfo if go.cgo_tools and go.mode.link in (LINKMODE_C_ARCHIVE, LINKMODE_C_SHARED): @@ -234,6 +238,13 @@ _go_binary_kwargs = { generated source files and static and generated embeddable files. """, ), + "env": attr.string_dict( + doc = """Environment variables to set when the binary is executed with bazel run. + The values (but not keys) are subject to + [location expansion](https://docs.bazel.build/versions/main/skylark/macros.html) but not full + [make variable expansion](https://docs.bazel.build/versions/main/be/make-variables.html). + """, + ), "importpath": attr.string( doc = """The import path of this binary. Binaries can't actually be imported, but this may be used by [go_path] and other tools to report the location of source |