[CI] Fixed Bugs in New Flow

Author: AlexandreSinger

.github/workflows/test.yml

@@ -34,42 +34,6 @@ env:
   MATRIX_EVAL: "CC=gcc-13 && CXX=g++-13"
 
 jobs:
-  Build:
-    name: 'B: Building VtR'
-    runs-on: ubuntu-24.04
-    strategy:
-      fail-fast: false
-      matrix:
-        include:
-        - { build_type: 'debug',   verbose: '0' }
-        - { build_type: 'debug',   verbose: '1' }
-    steps:
-
-    - uses: actions/setup-python@v5
-      with:
-        python-version: 3.12.3
-    - uses: actions/checkout@v4
-      with:
-        submodules: 'true'
-
-    - name: Get number of CPU cores
-      uses: SimenB/github-actions-cpu-cores@v2
-      id: cpu-cores
-
-    - name: Install dependencies
-      run: ./.github/scripts/install_dependencies.sh
-
-    - uses: hendrikmuhs/[email protected]
-
-    - name: Test
-      env:
-        BUILD_TYPE: ${{ matrix.build_type }}
-        NUM_PROC: ${{ steps.cpu-cores.outputs.count }}
-      run: |
-        export PATH="/usr/lib/ccache:/usr/local/opt/ccache/libexec:$PATH"
-        ./.github/scripts/build.sh VERBOSE=${{ matrix.verbose }}
-
-
   BuildVTR:
     name: 'B: Building VTR Release'
     runs-on: ubuntu-24.04
@@ -108,6 +72,7 @@ jobs:
         path: build.tar.gz
         retention-days: 1
 
+    # FIXME: This is not needed. Remove.
     - name: Print CCache Statistics
       run: |
         ccache -s
@@ -136,6 +101,7 @@ jobs:
         BUILD_TYPE: release
         NUM_PROC: ${{ steps.cpu-cores.outputs.count }}
       run: |
+        sudo sysctl -w vm.mmap_rnd_bits=28
         export PATH="/usr/lib/ccache:/usr/local/opt/ccache/libexec:$PATH"
         make -j${{ steps.cpu-cores.outputs.count}}
 
@@ -220,18 +186,15 @@ jobs:
     - name: Install dependencies
       run: ./.github/scripts/install_dependencies.sh
 
-    - name: Download Build Artifact
-      uses: actions/download-artifact@v4
-      with:
-        name: build-release.tar.gz
-
-    - name: Unpack Build
-      shell: bash
-      run: |
-        tar -xvzf build.tar.gz
+    - name: 'ccache'
+      uses: hendrikmuhs/[email protected]
 
     - name: Test
+      env:
+        CMAKE_PARAMS: "-DVTR_ASSERT_LEVEL=3"
+        NUM_PROC: ${{ steps.cpu-cores.outputs.count }}
       run: |
+        export PATH="/usr/lib/ccache:/usr/local/opt/ccache/libexec:$PATH"
         make test -j${{ steps.cpu-cores.outputs.count }}
 
 
@@ -240,6 +203,7 @@ jobs:
   # that these build variations are warning clean.
   BuildVariations:
     runs-on: ubuntu-24.04
+    needs: [BuildVTR]
     name: 'B: Build Variations'
     env:
       # For the CI, we want all build variations to be warning clean.
@@ -349,7 +313,7 @@ jobs:
             name: 'Strong',
             params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_IPO_BUILD=off -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on',
             suite: 'vtr_reg_strong',
-            extra_pkgs: "libeigen3-dev"
+            extra_pkgs: ""
           }
         ]
     name: 'R: ${{ matrix.name }}'
@@ -496,6 +460,7 @@ jobs:
 
   Sanitized:
     runs-on: ubuntu-24.04
+    needs: [BuildVTRWithOdin]
     strategy:
       fail-fast: false
       matrix:
@@ -654,6 +619,7 @@ jobs:
 
   Compatibility:
     runs-on: ubuntu-24.04
+    needs: [BuildVTR]
     strategy:
       fail-fast: false
       matrix:
@@ -697,6 +663,7 @@ jobs:
   JammyCompatibility:
     name: 'Ubuntu Jammy - 22.04 Compatibility Test'
     runs-on: ubuntu-22.04
+    needs: [BuildVTR]
     steps:
     - uses: actions/setup-python@v5
       with:
@@ -728,7 +695,6 @@ jobs:
   Coverity:
     name: 'Coverity Scan'
     needs:
-      - Build
       - Format
       - VerifyTestSuites
       - UnitTests

install_apt_packages.sh

@@ -13,7 +13,8 @@ sudo apt-get install -y \
 
 # Packages for more complex features of VTR that most people will use.
 sudo apt-get install -y \
-    libtbb-dev
+    libtbb-dev \
+    libeigen3-dev
 
 # Required for graphics
 sudo apt-get install -y \