[Openvpn-devel,v2] run-cppcheck.sh: Make CPPCHECK_CHECK_LEVEL=normal the default

Message ID 20261026120859.76788-1-frank@lichtenheld.com
State New
Headers
Series [Openvpn-devel,v2] run-cppcheck.sh: Make CPPCHECK_CHECK_LEVEL=normal the default |

Commit Message

Frank Lichtenheld Oct. 26, 2026, 12:08 p.m. UTC
  exhaustive is generally not usable since it runs for hours.
So avoid confusion by choosing a sane default.

Change-Id: Id206b12bd67c0b8a014a2c62277b43b18debfd43
Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
Acked-by: Razvan Cojocaru <razvanc@mailbox.org>
Acked-by: Arne Schwabe <arne-openvpn@rfc2549.org>
Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/2033
---

This change was reviewed on Gerrit and approved by at least one
developer. I request to merge it to master.

Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/2033
This mail reflects revision 2 of this Change.

Acked-by according to Gerrit (reflected above):
Razvan Cojocaru <razvanc@mailbox.org>
Arne Schwabe <arne-openvpn@rfc2549.org>
  

Comments

Gert Doering Oct. 26, 2026, 2:17 p.m. UTC | #1
Makes more sense than telling everbody "do not use the default, it breaks" ;-)

Your patch has been applied to the master branch.

commit 54a66f355ace8e89b25ed9e85c659d290eb700d2
Author: Frank Lichtenheld
Date:   Mon Oct 26 13:08:59 2026 +0100

     run-cppcheck.sh: Make CPPCHECK_CHECK_LEVEL=normal the default

     Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
     Acked-by: Razvan Cojocaru <razvanc@mailbox.org>
     Acked-by: Arne Schwabe <arne-openvpn@rfc2549.org>
     Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/2033
     Message-Id: <20261026120859.76788-1-frank@lichtenheld.com>
     URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg40139.html
     Signed-off-by: Gert Doering <gert@greenie.muc.de>


--
kind regards,

Gert Doering
  

Patch

diff --git a/dev-tools/run-cppcheck.sh b/dev-tools/run-cppcheck.sh
index a9c4643..07c0a6f 100755
--- a/dev-tools/run-cppcheck.sh
+++ b/dev-tools/run-cppcheck.sh
@@ -3,7 +3,7 @@ 
 set -eu
 
 : ${CPPCHECK_VERBOSE:=false}
-: ${CPPCHECK_CHECK_LEVEL:=exhaustive}
+: ${CPPCHECK_CHECK_LEVEL:=normal}
 
 if "${CPPCHECK_VERBOSE}"; then
     verbosity_arg="-v"