Skip to content

Use ijar:zipper instead of host zip/unzip for hermetic RBE builds - #541

Open
erikackermann wants to merge 1 commit into
bazelbuild:mainfrom
erikackermann:hermetic-ijar-zipper-rbe
Open

Use ijar:zipper instead of host zip/unzip for hermetic RBE builds#541
erikackermann wants to merge 1 commit into
bazelbuild:mainfrom
erikackermann:hermetic-ijar-zipper-rbe

Conversation

@erikackermann

Copy link
Copy Markdown

Summary

  • Point Android toolchain :zip / :unzip at @bazel_tools//third_party/ijar:zipper instead of host zip/unzip stubs
  • Add shared Starlark helpers and replace desugar genrules with rules that call zipper via ctx.actions.run
  • Update remaining shell call sites (create_system, databinding resource fix, build_java8_legacy_dex) to use zipper native x/v/c interface

Remote executors with --incompatible_strict_action_env do not provide host zip/unzip, which breaks Android builds on RBE.

Test plan

  • Build Android targets that exercise desugar config extraction and desugar_globals_jar
  • Build an android_library with databinding enabled

@google-cla

google-cla Bot commented Jul 28, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@erikackermann
erikackermann force-pushed the hermetic-ijar-zipper-rbe branch 7 times, most recently from 5ee30d6 to e34f22f Compare July 28, 2026 19:44
Route zip/unzip through @bazel_tools//third_party/ijar:zipper so actions
stay hermetic under --incompatible_strict_action_env. Resolve the real binary
via wrapper-first fallback to .runfiles/ (Linux RBE) or the wrapper on Windows.
@erikackermann
erikackermann force-pushed the hermetic-ijar-zipper-rbe branch from e34f22f to 54794d1 Compare July 28, 2026 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant