ci: Fix fail due to existent build dir

The second check is a "should fail" check, but it will fail because the build dir already exists rather than for the true reason. Use a different dir for the configure tests and move the `rm` inside the function. Signed-off-by: Lucas De Marchi <[email protected]>
kmod-project · Dec 5, 2024 · d6610b3 · d6610b3
1 parent a848bbf
commit d6610b3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -74,14 +74,14 @@ jobs:
         if: ${{ matrix.build == 'meson' }}
         run: |
           should_fail() {
-            if meson setup "$@" build/; then
+            rm -rf build-setup-test/
+            if meson setup "$@" build-setup-test/; then
               echo Command was expected to fail, but was successful
               return 1
             fi
           }
           should_fail -D distconfdir=relative/
           should_fail -D moduledir=relative/
-          rm -rf build/
 
       - name: configure (meson)
         if: ${{ matrix.build == 'meson' }}