Skip to content

Commit

Permalink
Allow overriding os_family in rbe_autoconfig
Browse files Browse the repository at this point in the history
Provides a workaround for bazelbuild#895.

Use like this:
rbe_autoconfig(
    name = "engflow_remote_config",
    os_family = "Linux",
    java_home = "/usr/lib/jvm/11.29.3-ca-jdk11.0.2/reduced",
    use_checked_in_confs = "Force",
)
  • Loading branch information
ulfjack authored and mattsoulanille committed Feb 11, 2021
1 parent 0b4a7f4 commit 4df3cf0
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
7 changes: 7 additions & 0 deletions rules/rbe_repo.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -823,6 +823,11 @@ _rbe_autoconfig = repository_rule(
doc = ("Optional. The Java release version in the container. For " +
" example, 11. Should only be set if java_home is set."),
),
"os_family": attr.string(
doc = ("Optional. The os_family to generate the config for. For example, " +
"Linux or Windows (Mac OS X is not supported at this time). The default is " +
"OS Bazel runs on."),
),
"registry": attr.string(
doc = ("Optional. The registry to pull the container from. For example, " +
"marketplace.gcr.io. The default is the value for the selected " +
Expand Down Expand Up @@ -912,6 +917,7 @@ def rbe_autoconfig(
export_configs = False,
java_home = None,
java_version = None,
os_family = None,
tag = None,
toolchain_config_suite_spec = default_toolchain_config_suite_spec(),
registry = None,
Expand Down Expand Up @@ -1244,6 +1250,7 @@ def rbe_autoconfig(
export_configs = export_configs,
java_home = java_home,
java_version = java_version,
os_family = os_family,
toolchain_config_suite_spec = toolchain_config_suite_spec_stripped,
registry = registry,
repository = repository,
Expand Down
2 changes: 2 additions & 0 deletions rules/rbe_repo/util.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,8 @@ def os_family(ctx):
Returns:
Returns the name of the OS Family
"""
if ctx.attr.os_family:
return ctx.attr.os_family
os_name = ctx.os.name.lower()
if os_name.find("windows") != -1:
return "Windows"
Expand Down

0 comments on commit 4df3cf0

Please sign in to comment.