From 17a216405540b1a2002cf86845ab99334f31513b Mon Sep 17 00:00:00 2001
From: Jonas Eschmann <jonas.eschmann@gmail.com>
Date: Sun, 26 May 2024 01:44:39 -0400
Subject: [PATCH] build

---
 .github/workflows/zoo.yml | 21 ++++++---------------
 1 file changed, 6 insertions(+), 15 deletions(-)

diff --git a/.github/workflows/zoo.yml b/.github/workflows/zoo.yml
index 80405c28..f7ff93d0 100644
--- a/.github/workflows/zoo.yml
+++ b/.github/workflows/zoo.yml
@@ -6,6 +6,7 @@ jobs:
   zoo:
     runs-on: ubuntu-latest
     steps:
+      - uses: actions/checkout@v3
       - name: Generate a token
         id: generate-token
         uses: actions/create-github-app-token@v1
@@ -20,7 +21,6 @@ jobs:
         run: |
           gh api octocat
           gh repo view rl-tools/zoo-blob --json name
-      - uses: actions/checkout@v3
 
       - name: Commit report
         run: |
@@ -45,25 +45,16 @@ jobs:
 #          source /opt/intel/oneapi/setvars.sh
 #          printenv >> $GITHUB_ENV
 
-      - name: Set reusable strings
-        # Turn repeated input strings (such as the build output directory) into step outputs. These step outputs can be used throughout the workflow file.
-        id: strings
-        shell: bash
-        run: |
-          echo "build-output-dir=${{ github.workspace }}/build" >> "$GITHUB_OUTPUT"
       - name: Configure CMake (Linux, macOS)
-        if: startsWith(matrix.os, 'ubuntu') || startsWith(matrix.os, 'macos')
         run: >
-          cmake -B ${{ steps.strings.outputs.build-output-dir }}
+          cmake -B build
           -DCMAKE_BUILD_TYPE=Release
-          -S ${{ github.workspace }}
+          -S .
           -DRL_TOOLS_ENABLE_TARGETS=ON
 #          -DRL_TOOLS_BACKEND_ENABLE_MKL=ON
       - name: Build
-        run: cmake --build ${{ steps.strings.outputs.build-output-dir }} --target rl_zoo
+        run: cmake --build build --target rl_zoo
 
-      - name: Test
+      - name: Run Zoo
         working-directory: ${{ github.workspace }}
-        # Execute tests defined by the CMake configuration. Note that --build-config is needed because the default Windows generator is a multi-config generator (Visual Studio generator).
-        # See https://cmake.org/cmake/help/latest/manual/ctest.1.html for more detail
-        run: ./${{ steps.strings.outputs.build-output-dir }}/src/rl/zoo/rl_zoo
+        run: ./build/src/rl/zoo/rl_zoo