[Openvpn-devel,XS] Change in openvpn[master]: Add --enable-werror to all platforms in Github Actions

Message ID 1e26c6425110ab7b2363fe9e87179e3d2df5e7db-HTML@gerrit.openvpn.net
State Superseded
Headers show
Series [Openvpn-devel,XS] Change in openvpn[master]: Add --enable-werror to all platforms in Github Actions | expand

Commit Message

reynir (Code Review) Oct. 18, 2023, 5:21 p.m. UTC
Attention is currently required from: flichtenheld.

Hello flichtenheld,

I'd like you to do a code review.
Please visit

    http://gerrit.openvpn.net/c/openvpn/+/376?usp=email

to review the following change.


Change subject: Add --enable-werror to all platforms in Github Actions
......................................................................

Add --enable-werror to all platforms in Github Actions

Change-Id: I8f06a1213fdca233671f8d5746216ae46e84233b
---
M .github/workflows/build.yaml
1 file changed, 3 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/76/376/1

Patch

diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
index 2a25582..4344121 100644
--- a/.github/workflows/build.yaml
+++ b/.github/workflows/build.yaml
@@ -186,7 +186,7 @@ 
       - name: autoconf
         run: autoreconf -fvi
       - name: configure
-        run: CFLAGS="-fsanitize=address,undefined -fno-sanitize-recover=all  -fno-omit-frame-pointer -O2" CC=clang ./configure --with-crypto-library=${{matrix.ssllib}}
+        run: CFLAGS="-fsanitize=address,undefined -fno-sanitize-recover=all  -fno-omit-frame-pointer -O2" CC=clang ./configure --with-crypto-library=${{matrix.ssllib}} --enable-werror
       - name: make all
         run: make -j3
       - name: make check
@@ -235,7 +235,7 @@ 
       - name: autoconf
         run: autoreconf -fvi
       - name: configure
-        run: ./configure --enable-werror ${{matrix.configureflags}} ${{matrix.configuressllib}}
+        run: ./configure --enable-werror ${{matrix.configureflags}} ${{matrix.configuressllib}} --enable-werror
       - name: make all
         run: make -j4
       - name: make check
@@ -355,7 +355,7 @@ 
       - name: autoconf
         run: autoreconf -fvi
       - name: configure
-        run: ./configure --with-crypto-library=openssl ${{matrix.configureflags}}
+        run: ./configure --with-crypto-library=openssl ${{matrix.configureflags}} --enable-werror
       - name: make all
         run: make -j3
       - name: make check