Merge pull request #106 from arkedge/make-gcc-optional

Make gcc skip optional instead of skip gcc -Werror
arkedge · Jul 22, 2024 · ce3b15d · ce3b15d
2 parents d3c3b68 + 28e4da1
commit ce3b15d
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 4 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -18,6 +18,10 @@ on:
       sils_mockup:
         type: boolean
         default: false
+      # skip gcc from build to support C89-like target
+      skip_gcc:
+        type: boolean
+        default: false
       build_msvc:
         type: boolean
         default: false
@@ -72,17 +76,16 @@ jobs:
       matrix:
         compiler: [gcc, clang]
         warning: [Werror, Wextra]
-        exclude:
-          - compiler: gcc
-            warning: Werror
 
     steps:
       - uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
         with:
           repository: arkedge/workflows-c2a
           ref: ${{ env.SELF_VERSION }}
 
-      - uses: ./action-c2a-build
+      - name: build
+        if: matrix.compiler != 'gcc' || (! inputs.skip_gcc)
+        uses: ./action-c2a-build
         env:
           CC: ${{ matrix.compiler }}
         with:

diff --git a/.github/workflows/default-v3.yml b/.github/workflows/default-v3.yml
@@ -9,6 +9,10 @@ on:
       c2a_dir:
         type: string
         default: .
+      # skip gcc from build to support C89-like target
+      skip_gcc:
+        type: boolean
+        default: false
       build_msvc:
         type: boolean
         default: false
@@ -29,6 +33,7 @@ jobs:
     with:
       federation_repos: ${{ inputs.federation_repos }}
       c2a_dir: ${{ inputs.c2a_dir }}
+      skip_gcc: ${{ inputs.skip_gcc }}
       build_msvc: ${{ inputs.build_msvc }}
       build_as_cxx: ${{ inputs.build_as_cxx }}
       reviewdog_version: ${{ inputs.reviewdog_version }}

diff --git a/.github/workflows/default.yml b/.github/workflows/default.yml
@@ -9,6 +9,10 @@ on:
       c2a_dir:
         type: string
         default: .
+      # skip gcc from build to support C89-like target
+      skip_gcc:
+        type: boolean
+        default: false
       build_msvc:
         type: boolean
         default: false
@@ -29,6 +33,7 @@ jobs:
     with:
       federation_repos: ${{ inputs.federation_repos }}
       c2a_dir: ${{ inputs.c2a_dir }}
+      skip_gcc: ${{ inputs.skip_gcc }}
       build_msvc: ${{ inputs.build_msvc }}
       build_as_cxx: ${{ inputs.build_as_cxx }}
       reviewdog_version: ${{ inputs.reviewdog_version }}

diff --git a/.github/workflows/test-build.yml b/.github/workflows/test-build.yml
@@ -29,6 +29,7 @@ jobs:
       cmake_generator_linux32: Ninja
       cmake_flags_linux32: -DUSE_SCI_COM_WINGS=OFF -DC2A_USE_SCI_COM_WINGS=OFF
       sils_mockup: true
+      skip_gcc: true
       build_as_cxx: true
       reviewdog_default_reporter: github-check
       reviewdog_default_filter: added