Skip to content

Commit

Permalink
buildbuddy.yaml: Add and invoke //:checkout_modules sh_binary tar…
Browse files Browse the repository at this point in the history
…get to support submodules.
  • Loading branch information
ivucica committed Jun 11, 2024
1 parent ba8a59f commit a825adb
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 0 deletions.
5 changes: 5 additions & 0 deletions BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -1153,6 +1153,11 @@ genrule(
# ==============
# end from envoy

sh_binary(
name = "checkout_modules",
srcs = ["_checkout_modules.sh"],
)

genrule(
name = "stamp_cpp",
outs = ["stamp.cpp"],
Expand Down
5 changes: 5 additions & 0 deletions _checkout_modules.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
#!/bin/bash

# For use with CI that does not support git submodules (e.g. if it needs to be run with bazel run).
git describe
git submodule update --init
2 changes: 2 additions & 0 deletions buildbuddy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ actions:
branches:
- "*"
bazel_commands:
- "run --config=remote :checkout_modules"
- "test --config=remote //:util_test" # <-- main modification here: --config=remote as a default for buildbuddy.yaml; also currently testing even //:all is broken (because it tries to compile macclipboard.m), much less //...

- name: "Build main binary"
Expand All @@ -20,4 +21,5 @@ actions:
branches:
- "*"
bazel_commands:
- "run --config=remote :checkout_modules"
- "build --config=remote :yatc"

0 comments on commit a825adb

Please sign in to comment.