Skip to content

Commit

Permalink
CI/modules-zstd: Fix calls to multiple bazel run targets
Browse files Browse the repository at this point in the history
bazel run executes only the first target from the list of targets acquired
from the output of bazel query. In order to properly call all targets
it is required to loop through the targets and run one at a time

Signed-off-by: Pawel Czarnecki <[email protected]>
  • Loading branch information
lpawelcz committed Oct 4, 2024
1 parent 87567ce commit 94a6d6d
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions .github/workflows/modules-zstd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,12 +60,20 @@ jobs:
- name: Build and run ZSTD IR benchmark rules (opt)
if: ${{ !cancelled() }}
run: |
bazel run -c opt -- $(bazel query 'filter(".*_ir_benchmark", kind(rule, //xls/modules/zstd/...))')
for target in $(bazel query 'filter(".*_ir_benchmark$", kind(rule, //xls/modules/zstd/...))');
do
echo "running $target";
bazel run -c opt $target -- --logtostderr;
done
- name: Build and run synthesis benchmarks of the ZSTD module (opt)
if: ${{ !cancelled() }}
run: |
bazel run -c opt -- $(bazel query 'filter(".*_benchmark_synth", kind(rule, //xls/modules/zstd/...))')
for target in $(bazel query 'filter(".*_benchmark_synth$", kind(rule, //xls/modules/zstd/...))');
do
echo "running $target";
bazel run -c opt $target -- --logtostderr;
done
- name: Build ZSTD place and route targets (opt)
if: ${{ !cancelled() }}
Expand Down

0 comments on commit 94a6d6d

Please sign in to comment.