From bf1bad352864acd6b4c7c74d6a3b97511051fe75 Mon Sep 17 00:00:00 2001 From: Kunshan Wang Date: Thu, 14 Dec 2023 14:44:20 +0800 Subject: [PATCH] Fix Ruby binding test repo checkout (#1048) The Ruby binding test now checks out the revision of the `ruby` repo selected in `mmtk-ruby/mmtk/Cargo.toml` instead of the latest revision. --- .github/workflows/post-review-ci.yml | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/.github/workflows/post-review-ci.yml b/.github/workflows/post-review-ci.yml index 3957caf8ab..e87c4bbe43 100644 --- a/.github/workflows/post-review-ci.yml +++ b/.github/workflows/post-review-ci.yml @@ -192,12 +192,21 @@ jobs: path: mmtk-ruby ref: ${{ needs.binding-refs.outputs.ruby_binding_ref }} + - name: Setup script dependencies + run: ./.github/scripts/ci-setup-script-deps.sh + working-directory: mmtk-ruby + + - name: Determine Ruby repository and revision + id: extract-ruby-revision + run: python ./.github/scripts/extract-ruby-revision.py ./mmtk/Cargo.toml --github-output=$GITHUB_OUTPUT + working-directory: mmtk-ruby + - name: Checkout Ruby uses: actions/checkout@v3 with: - repository: mmtk/ruby + repository: ${{ steps.extract-ruby-revision.outputs.ruby_repo }} + ref: ${{ steps.extract-ruby-revision.outputs.ruby_rev }} path: ruby - ref: mmtk - name: Override mmtk-core dependency for binding run: ./.github/scripts/ci-replace-mmtk-dep.sh ../mmtk-ruby/mmtk/Cargo.toml --mmtk-core-path .