Skip to content

Commit 9cbf80d

Browse files
committed
[bazel] Do fusesoc builds in the sandbox
Fix up deps and do the fusesoc builds in the sandbox. Since the Python interpreter was switched to the bazel-provided one, it is now possible to use the sandbox. Note that this doesn't make the build *hermetic*, since it still relies on system-provided tools (e.g. vivado, verilator). Signed-off-by: Alexander Williams <[email protected]>
1 parent b0ba319 commit 9cbf80d

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ package(default_visibility = ["//visibility:public"])
77
exports_files([
88
"WORKSPACE",
99
"python-requirements.txt",
10+
"tool_requirements.py",
1011
])
1112

1213
filegroup(

hw/BUILD

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,10 @@ filegroup(
111111
# TODO(lowRISC/opentitan#15882): make Verilator work with foundry repo present.
112112
exclude = ["foundry/**"],
113113
) + [
114+
"//:tool_requirements.py",
114115
"//hw/ip:all_files",
115116
"//hw/top_earlgrey:all_files",
117+
"//util:check_tool_requirements.py",
116118
],
117119
visibility = ["//visibility:public"],
118120
)

rules/fusesoc.bzl

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,9 +76,6 @@ def _fusesoc_build_impl(ctx):
7676
arguments = [args],
7777
executable = ctx.executable._fusesoc,
7878
use_default_shell_env = False,
79-
execution_requirements = {
80-
"no-sandbox": "",
81-
},
8279
env = ENV,
8380
)
8481
return [

0 commit comments

Comments
 (0)