github bot: remove explicit flags

The explicit flags passed to the build system are identical or more restricitve than the defaults breaking for example builds using clang. Removing the explicit flags mean we do not have to maintain multiple sets of compiler flags. Signed-off-by: Florian Fischer <[email protected]>
axboe · Jul 4, 2022 · 9d93f5d · 9d93f5d
1 parent 34b3e45
commit 9d93f5d
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -91,9 +91,6 @@ jobs:
             cc: mips-linux-gnu-gcc
             cxx: mips-linux-gnu-g++
 
-    env:
-      FLAGS: -g -O2 -Wall -Wextra -Werror
-
     steps:
     - name: Checkout source
       uses: actions/checkout@v2
@@ -139,21 +136,21 @@ jobs:
       run: |
         if [[ "${{matrix.arch}}" == "x86_64" ]]; then \
           if [[ "${{matrix.cc}}" == "clang" ]]; then \
-            make CC=${{matrix.cc}} CXX=${{matrix.cxx}} CPPFLAGS="-Werror" CFLAGS="$FLAGS" CXXFLAGS="$FLAGS"; \
+            make CC=${{matrix.cc}} CXX=${{matrix.cxx}}; \
           else \
 
-            make CPPFLAGS="-Werror" CFLAGS="$FLAGS" CXXFLAGS="$FLAGS"; \
+            make;\
           fi; \
         else \
-           CPPFLAGS="-Werror" CFLAGS="$FLAGS" CXXFLAGS="$FLAGS" meson build --cross-file /tmp/cross-env.txt; \
+           meson build --cross-file /tmp/cross-env.txt; \
            ninja -C build -j$(nproc); \
         fi;
 
     - name: Build nolibc
       run: |
         if [[ "${{matrix.arch}}" == "x86_64" || "${{matrix.arch}}" == "i686" ]]; then \
             make clean; \
-            CC=${{matrix.cc}} CXX=${{matrix.cxx}} CPPFLAGS="-Werror" CFLAGS="$FLAGS" CXXFLAGS="$FLAGS" meson setup build -Dnolibc=true -Dtests=true -Dexamples=true; \
+            CC=${{matrix.cc}} CXX=${{matrix.cxx}} meson setup build -Dnolibc=true -Dtests=true -Dexamples=true; \
             ninja -C build -j$(nproc); \
         else \
             echo "Skipping nolibc build, this arch doesn't support building liburing without libc"; \