skia2/experimental
Kevin Lubick a74ebc9bac [bazel] Use exec_tools instead of tools for better RBE compatibility
The tools in genrules [1] correspond to the HOST's version, which
is a problem if we are doing cross-platform builds. In this case,
when I was using my M1 Mac to run on a Linux RBE platform, the
Mac's python binaries (e.g. python3_9_aarch64-apple-darwin/bin/python3)
were being uploaded to RBE and attempted to be used, which resulted
in OSError: [Errno 8] Exec format error because the won't run
on the Linux RBE machine.

Thanks to tjgq@, I learned about exec_tools [2], which will use
the version of the tools for the EXECUTION platform, which
is exactly what we want.

While debugging this, I added a minimal reproduction case in
//experimental/bazel_test and updated the py_tools version
in an effort to diagnose the issue further.

We will need to contribute similar fixes to @spirv_tools

[1] https://bazel.build/reference/be/general#genrule.tools
[2] https://bazel.build/reference/be/general#genrule.exec_tools
Bug: skia:12541
Change-Id: Ib14deb4e326d3103fd08c21e93afe342d751c17a
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/554518
Reviewed-by: Jorge Betancourt <jmbetancourt@google.com>
2022-06-30 18:02:35 +00:00
..
bazel_test [bazel] Use exec_tools instead of tools for better RBE compatibility 2022-06-30 18:02:35 +00:00
c-api-example Fix C API example, ensure the C API example is built in CI 2020-12-15 16:38:42 +00:00
docs runs some sample animations side by side in canvas and svg 2015-11-16 13:36:08 -08:00
documentation Update references to master branch 2021-05-21 15:18:16 +00:00
ffmpeg [bazel] Delete gazelle-based BUILD.bazel files 2022-05-31 14:14:54 +00:00
lowp-basic add constrained_add 2021-11-01 17:30:16 +00:00
minimal_ios_mtl_skia_app [includes] Prepare for moving SkColorSpace to forward declare 2022-03-31 19:50:10 +00:00
sktext Cleanup SkText warnings 2022-06-01 13:03:05 +00:00
sorttoy Move all Ganesh source files into ganesh subdirectory. 2022-04-07 21:06:50 +00:00
tools Reland "[infra] Remove old python scripts and urllib2 references" 2022-05-16 20:15:49 +00:00
tskit [infra] Deduplicate serve.py 2021-10-12 11:42:50 +00:00
wasm-hello-world [bazel] Add "skia_internal" target that exposes private API for tests/tools. 2022-06-02 11:23:50 +00:00
webgpu-bazel [bazel] Add "skia_internal" target that exposes private API for tests/tools. 2022-06-02 11:23:50 +00:00