]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
checksrc: avoid extra runs in CI, enable more check locally, fix fallouts
authorViktor Szakats <commit@vsz.me>
Sat, 17 May 2025 00:02:23 +0000 (02:02 +0200)
committerViktor Szakats <commit@vsz.me>
Mon, 19 May 2025 16:59:50 +0000 (18:59 +0200)
To avoid redundant work in CI and to avoid a single checksrc issue make
all autotools jobs fail. After this patch checksrc issues make fail
the checksrc job, the `dist / verify-out-of-tree-autotools-debug`,
`dist / maketgz-and-verify-in-tree`  jobs and the fuzzer job (if run).
Of these, the `dist` jobs replicate local builds, also testing the build
logic.

Also add a script to check the complete local repository, optionally
with the build tree to verify generated C files.

Also:
- automatically run checksrc in subdirectories having a `checksrc`
  target. (examples, OS400, tests http/client, unit and tunit)
- tests/libtest: make sure to run `checksrc` on generated `lib1521.c`.
  (requires in-tree autotools build.)
- tests: run `checksrc` on targets also for non-`DEBUGBUILD`
  builds. It ensures to check `lib1521.c` in CI via job
  `dist / maketgz-and-verify-in-tree`.
- src: drop redundant `$(builddir)` in autotools builds.
- scripts: add `checksrc-all.sh` script to check all C sources and
  the build directory as an option.
- use the above from CI, also make it verify all generated sources.
- silence `checksrc` issues in generated C sources.
- checksrc: add `-v` option to enable verbose mode.
- checksrc: make verbose mode show checked filename and fix to only
  return error on failure.
- make sure that generated C files pass `checksrc`.

Assisted-by: Daniel Stenberg
Closes #17376

26 files changed:
.github/workflows/checksrc.yml
.github/workflows/distcheck.yml
.github/workflows/http3-linux.yml
.github/workflows/linux-old.yml
.github/workflows/linux.yml
.github/workflows/macos.yml
.github/workflows/non-native.yml
.github/workflows/windows.yml
configure.ac
docs/cmdline-opts/pinnedpubkey.md
docs/examples/Makefile.am
include/curl/Makefile.am
lib/Makefile.am
packages/Makefile.am
scripts/Makefile.am
scripts/checksrc-all.sh [new file with mode: 0755]
scripts/checksrc.pl
src/CMakeLists.txt
src/Makefile.am
src/mk-file-embed.pl
tests/http/clients/Makefile.am
tests/libtest/Makefile.am
tests/mk-bundle.pl
tests/server/Makefile.am
tests/tunit/Makefile.am
tests/unit/Makefile.am

index 4083cc3c569fea81ce675dc030b9812e5cd1e8b2..990551aeab86a41c4960c328419a023cbc2c9bee 100644 (file)
@@ -41,7 +41,7 @@ jobs:
         name: checkout
 
       - name: check
-        run: git ls-files -z "*.[ch]" | xargs -0 -n1 ./scripts/checksrc.pl
+        run: scripts/checksrc-all.sh
 
   codespell-cmakelint-pytype-ruff:
     runs-on: ubuntu-latest
index 20d4a7aa3d910d8a5d0c2862916358f6e0bce1cc..c851de0cd33f2500456ffdfaa9c1757d67cbac05 100644 (file)
@@ -109,6 +109,8 @@ jobs:
           make
           make test-ci
           make install
+          popd
+          scripts/checksrc-all.sh
 
   verify-out-of-tree-cmake:
     runs-on: ubuntu-latest
index 80cae92d431318f8b0f8d91c2b5e1a4d0234b2c1..fd1ecbcd54acf34a31ac71463d43dba24318280b 100644 (file)
@@ -38,6 +38,7 @@ permissions: {}
 
 env:
   MAKEFLAGS: -j 5
+  CURL_CI: github
   # handled in renovate.json
   openssl-version: 3.5.0
   # handled in renovate.json
@@ -527,7 +528,6 @@ jobs:
       - name: 'run pytest event based'
         env:
           CURL_TEST_EVENT: 1
-          CURL_CI: github
           PYTEST_ADDOPTS: '--color=yes'
           PYTEST_XDIST_AUTO_NUM_WORKERS: 4
         run: |
index 8ad3bc00d345015f900d2adab09fb205587bf0c2..6d7c6a8b390e73ab271ef61604276d7233e9749b 100644 (file)
@@ -47,6 +47,7 @@ permissions: {}
 
 env:
   MAKEFLAGS: -j 5
+  CURL_CI: github
   DEBIAN_FRONTEND: noninteractive
 
 jobs:
index 52328ba478ad6c101aeb48654ded4f937779a39c..295d8a637816baad7163f7ac046b2bf2eedad24c 100644 (file)
@@ -37,6 +37,7 @@ permissions: {}
 
 env:
   MAKEFLAGS: -j 5
+  CURL_CI: github
   CURL_CLANG_TIDYFLAGS: '-checks=-clang-analyzer-security.insecureAPI.strcpy,-clang-analyzer-optin.performance.Padding,-clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling,-clang-analyzer-valist.Uninitialized'
   # unhandled
   bearssl-version: 0.6
@@ -698,7 +699,6 @@ jobs:
       - name: 'run pytest'
         if: contains(matrix.build.install_steps, 'pytest')
         env:
-          CURL_CI: github
           PYTEST_ADDOPTS: '--color=yes'
           PYTEST_XDIST_AUTO_NUM_WORKERS: 4
         run: |
index 0f619cee92c2278212fdb8b2a18cc26d2e881975..c4e558bb117bafdb6631ce0625c9149e1890ddc9 100644 (file)
@@ -47,6 +47,7 @@ permissions: {}
 # newer than the 10.8 required by `CFURLCreateDataAndPropertiesFromResource`.
 
 env:
+  CURL_CI: github
   MAKEFLAGS: -j 4
   LDFLAGS: -w  # suppress 'object file was built for newer macOS version than being linked' warnings
 
@@ -338,7 +339,6 @@ jobs:
       - name: 'run pytest'
         if: ${{ !matrix.build.clang-tidy && contains(matrix.build.install_steps, 'pytest') }}
         env:
-          CURL_CI: github
           PYTEST_ADDOPTS: '--color=yes'
           PYTEST_XDIST_AUTO_NUM_WORKERS: 4
         run: |
index 64dde730d54aedf7f1eda35068a16791e3fe897b..eb49a87010b3b462d62700574842beb214e65ed1 100644 (file)
@@ -35,6 +35,9 @@ concurrency:
 
 permissions: {}
 
+env:
+  CURL_CI: github
+
 jobs:
   netbsd:
     name: 'NetBSD, CM clang openssl ${{ matrix.arch }}'
@@ -141,6 +144,7 @@ jobs:
           architecture: ${{ matrix.arch }}
           run: |
             export MAKEFLAGS=-j3
+            export CURL_CI=github
             # https://ports.freebsd.org/
             time sudo pkg install -y autoconf automake libtool \
               pkgconf brotli openldap26-client libidn2 libnghttp2 stunnel py311-impacket
@@ -229,6 +233,7 @@ jobs:
             set -e
             ln -s /usr/bin/gcpp /usr/bin/cpp  # Some tests expect `cpp`, which is named `gcpp` in this env.
             export MAKEFLAGS=-j3
+            export CURL_CI=github
             time autoreconf -fi
             mkdir bld && cd bld && time ../configure --enable-unity --enable-test-bundles --enable-debug --enable-warnings --enable-werror \
               --prefix="${HOME}"/install \
index 0a8b813025924d105ce819731d28a23d04499df7..f4b2ad50e44086e55483a30b32921ac6a86e2607 100644 (file)
@@ -35,6 +35,9 @@ concurrency:
 
 permissions: {}
 
+env:
+  CURL_CI: github
+
 jobs:
   cygwin:
     name: "cygwin, ${{ matrix.build == 'cmake' && 'CM' || 'AM' }} ${{ matrix.platform }} ${{ matrix.name }}"
index dd88ea5e306adc495bc1dfe9d7f8c8787f22dae4..8a91594a5cf46d62bdba0ecd89d19a828247ad65 100644 (file)
@@ -39,6 +39,7 @@ terms of the curl license; see COPYING for more details])
 
 AC_CONFIG_SRCDIR([lib/urldata.h])
 AC_CONFIG_HEADERS(lib/curl_config.h)
+AH_TOP([/* !checksrc! disable COPYRIGHT all */])
 AC_CONFIG_MACRO_DIR([m4])
 AM_MAINTAINER_MODE
 m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])])
@@ -453,6 +454,8 @@ XC_LIBTOOL
 
 LT_LANG([Windows Resource])
 
+AM_CONDITIONAL(NOT_CURL_CI, test -z "$CURL_CI")
+
 #
 # Automake conditionals based on libtool related checks
 #
index d21a18f69373c5e047989a46776016cde4a50a6f..fab43456c504a58027dd578a1c495e8061e40a94 100644 (file)
@@ -32,8 +32,9 @@ together then the peer is still verified by public key.
 
 PEM/DER support:
 
-OpenSSL and GnuTLS (added in 7.39.0), wolfSSL (added in 7.43.0), mbedTLS
-(added in 7.47.0), Secure Transport macOS 10.7+/iOS 10+ (added in 7.54.1),
+OpenSSL and GnuTLS (added in 7.39.0), wolfSSL (added in 7.43.0),
+mbedTLS (added in 7.47.0),
+Secure Transport macOS 10.7+/iOS 10+ (added in 7.54.1),
 Schannel (added in 7.58.1)
 
 sha256 support:
index 02ba20b99fb7b3c8e0d59bc03e72579e2fdbf3c0..042922f1c4e21af0bfe6874de982cd52d0c9a9bb 100644 (file)
@@ -69,3 +69,10 @@ CS_ = $(CS_0)
 
 checksrc:
        $(CHECKSRC)(@PERL@ $(top_srcdir)/scripts/checksrc.pl -D$(srcdir) $(srcdir)/*.c)
+
+if NOT_CURL_CI
+if DEBUGBUILD
+# for debug builds, we scan the sources on all regular make invokes
+all-local: checksrc
+endif
+endif
index 8864d60e1f2abcb40fe2a4a9549b51392a7a27b0..f664f6ab7ef5fe43bd116a5c3eb4cb317a804b6b 100644 (file)
@@ -35,7 +35,9 @@ CS_ = $(CS_0)
 checksrc:
        $(CHECKSRC)@PERL@ $(top_srcdir)/scripts/checksrc.pl -D$(top_srcdir)/include/curl $(pkginclude_HEADERS)
 
+if NOT_CURL_CI
 if DEBUGBUILD
 # for debug builds, we scan the sources on all regular make invokes
 all-local: checksrc
 endif
+endif
index 9c26d4d6674ad222e749091312f1ce26e21c83e4..8dac236606f5cff77f5a8931fec5e7f4c91b4ab4 100644 (file)
@@ -163,10 +163,12 @@ checksrc:
        $(CHECKSRC)(@PERL@ $(top_srcdir)/scripts/checksrc.pl -D$(srcdir) \
        $(CSOURCES) $(HHEADERS))
 
+if NOT_CURL_CI
 if DEBUGBUILD
 # for debug builds, we scan the sources on all regular make invokes
 all-local: checksrc
 endif
+endif
 
 # disable the tests that are mostly causing false positives
 TIDYFLAGS := -checks=-clang-analyzer-security.insecureAPI.strcpy,-clang-analyzer-optin.performance.Padding,-clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling -quiet
index 96f52bdb4c866c052d678af90baa22ecc029410e..43e544a0195c3951b309b3eafb3f02630d26fd4c 100644 (file)
@@ -50,3 +50,10 @@ CS_ = $(CS_0)
 
 checksrc:
        $(CHECKSRC)(@PERL@ $(top_srcdir)/scripts/checksrc.pl -D$(srcdir) $(srcdir)/OS400/*.[ch])
+
+if NOT_CURL_CI
+if DEBUGBUILD
+# for debug builds, we scan the sources on all regular make invokes
+all-local: checksrc
+endif
+endif
index 1ad250ae311f50f8ecd36898bc78de4a5509742f..03eb8044ad67c45bcad92e53d769aebc534d0c29 100644 (file)
@@ -22,9 +22,9 @@
 #
 ###########################################################################
 
-EXTRA_DIST = coverage.sh completion.pl firefox-db2pem.sh checksrc.pl              \
-  mk-ca-bundle.pl mk-unity.pl schemetable.c cd2nroff nroff2cd cdall cd2cd managen \
-  dmaketgz maketgz release-tools.sh verify-release cmakelint.sh mdlinkcheck       \
+EXTRA_DIST = coverage.sh completion.pl firefox-db2pem.sh checksrc.pl checksrc-all.sh \
+  mk-ca-bundle.pl mk-unity.pl schemetable.c cd2nroff nroff2cd cdall cd2cd managen    \
+  dmaketgz maketgz release-tools.sh verify-release cmakelint.sh mdlinkcheck          \
   CMakeLists.txt pythonlint.sh randdisable wcurl
 
 dist_bin_SCRIPTS = wcurl
diff --git a/scripts/checksrc-all.sh b/scripts/checksrc-all.sh
new file mode 100755 (executable)
index 0000000..2c4b3c5
--- /dev/null
@@ -0,0 +1,23 @@
+#!/bin/sh
+# Copyright (C) Viktor Szakats
+#
+# SPDX-License-Identifier: curl
+
+set -eu
+
+anyfailed=0
+
+for dir in $({
+    if git rev-parse --is-inside-work-tree >/dev/null 2>&1; then
+      git ls-files '*.[ch]'
+    else
+      find . -name '*.[ch]'
+    fi
+    [ -n "${1:-}" ] && find "$@" -name '*.[ch]'
+  } | grep -v -F '/CMakeFiles/' | sed -E 's|/[^/]+$||' | sort -u); do
+  if ! ./scripts/checksrc.pl "${dir}"/*.[ch]; then
+    anyfailed=1
+  fi
+done
+
+exit "${anyfailed}"
index 68189b376fefd61264bbc1dbb117b4b5e83c3a11..f62962ff1aca37221328e2853dbc566bfa68220c 100755 (executable)
@@ -39,7 +39,7 @@ my $dir=".";
 my $wlist="";
 my @alist;
 my $windows_os = $^O eq 'MSWin32' || $^O eq 'cygwin' || $^O eq 'msys';
-my $verbose;
+my $verbose = 0;
 my %skiplist;
 
 my %ignore;
@@ -288,6 +288,11 @@ while(defined $file) {
         $file = shift @ARGV;
         next;
     }
+    elsif($file =~ /^-v/) {
+        $verbose = 1;
+        $file = shift @ARGV;
+        next;
+    }
     elsif($file =~ /^(-h|--help)/) {
         undef $file;
         last;
@@ -307,6 +312,7 @@ if(!$file) {
     print "  -W[file]  Skip the given file - ignore all its flaws\n";
     print "  -i<n>     Indent spaces. Default: 2\n";
     print "  -m<n>     Maximum line length. Default: 79\n";
+    print "  -v        Verbose\n";
     print "\nDetects and warns for these problems:\n";
     my @allw = keys %warnings;
     push @allw, keys %warnings_extended;
@@ -448,6 +454,11 @@ sub scanfile {
     my $l = "";
     my $prep = 0;
     my $prevp = 0;
+
+    if($verbose) {
+        printf "Checking file: $file\n";
+    }
+
     open(my $R, '<', $file) || die "failed to open $file";
 
     my $incomment=0;
@@ -1123,5 +1134,7 @@ if($errors || $warnings || $verbose) {
         $serrors,
         $swarnings;
     }
-    exit 5; # return failure
+    if($errors || $warnings) {
+        exit 5; # return failure
+    }
 }
index 469df6d7611d31fb9ba2aed746cd6f7069ec9578..9cae87b01dbd1b75f14d30f0da7c1b2fb5c52bb4 100644 (file)
@@ -33,6 +33,9 @@ if(ENABLE_CURL_MANUAL AND HAVE_MANUAL_TOOLS)
   add_custom_command(
     OUTPUT "tool_hugehelp.c"
     COMMAND ${CMAKE_COMMAND} -E echo "#include \"tool_setup.h\"" > "tool_hugehelp.c"
+    COMMAND ${CMAKE_COMMAND} -E echo "/* !checksrc! disable COPYRIGHT all */" >> "tool_hugehelp.c"
+    COMMAND ${CMAKE_COMMAND} -E echo "/* !checksrc! disable INCLUDEDUP all */" >> "tool_hugehelp.c"
+    COMMAND ${CMAKE_COMMAND} -E echo "/* !checksrc! disable LONGLINE all */" >> "tool_hugehelp.c"
     COMMAND ${CMAKE_COMMAND} -E echo "#ifndef HAVE_LIBZ" >> "tool_hugehelp.c"
     COMMAND "${PERL_EXECUTABLE}" "${CMAKE_CURRENT_SOURCE_DIR}/mkhelp.pl"    < "${CURL_ASCIIPAGE}" >> "tool_hugehelp.c"
     COMMAND ${CMAKE_COMMAND} -E echo "#else" >> "tool_hugehelp.c"
index 5e84a20b14d14eaca6e5f796650be8f1dfacad43..2df95a0f94297199a032457c4034ee3575080488 100644 (file)
@@ -125,7 +125,7 @@ endif
 # Use absolute directory to disable VPATH
 ASCIIPAGE=$(top_builddir)/docs/cmdline-opts/curl.txt
 MKHELP=$(top_srcdir)/src/mkhelp.pl
-HUGE=$(builddir)/tool_hugehelp.c
+HUGE=tool_hugehelp.c
 
 HUGECMD = $(HUGEIT_$(V))
 HUGEIT_0 = @echo "  HUGE    " $@;
@@ -145,22 +145,29 @@ if HAVE_LIBZ
 # This generates the tool_hugehelp.c file in both uncompressed and
 # compressed formats.
 $(HUGE): $(ASCIIPAGE) $(MKHELP)
-       $(HUGECMD) (echo '#include "tool_setup.h"' > $(HUGE);   \
-       echo '#ifndef HAVE_LIBZ' >> $(HUGE);                    \
-       $(PERL) $(MKHELP) < $(ASCIIPAGE) >> $(HUGE);     \
-       echo '#else' >> $(HUGE);                                \
-       $(PERL) $(MKHELP) -c < $(ASCIIPAGE) >> $(HUGE);  \
+       $(HUGECMD)(                                                \
+       echo '/* !checksrc! disable COPYRIGHT all */' > $(HUGE);   \
+       echo '/* !checksrc! disable INCLUDEDUP all */' >> $(HUGE); \
+       echo '/* !checksrc! disable LONGLINE all */' >> $(HUGE);   \
+       echo '#include "tool_setup.h"' >> $(HUGE);                 \
+       echo '#ifndef HAVE_LIBZ' >> $(HUGE);                       \
+       $(PERL) $(MKHELP) < $(ASCIIPAGE) >> $(HUGE);               \
+       echo '#else' >> $(HUGE);                                   \
+       $(PERL) $(MKHELP) -c < $(ASCIIPAGE) >> $(HUGE);            \
        echo '#endif /* HAVE_LIBZ */' >> $(HUGE) )
 else # HAVE_LIBZ
 # This generates the tool_hugehelp.c file uncompressed only
 $(HUGE): $(ASCIIPAGE) $(MKHELP)
-       $(HUGECMD)(echo '#include "tool_setup.h"' > $(HUGE);    \
+       $(HUGECMD)(                                                \
+       echo '/* !checksrc! disable COPYRIGHT all */' > $(HUGE);   \
+       echo '#include "tool_setup.h"' >> $(HUGE);                 \
        $(PERL) $(MKHELP) < $(ASCIIPAGE) >> $(HUGE) )
 endif
 
 else # USE_MANUAL
 # built-in manual has been disabled, make a blank file
 $(HUGE):
+       echo '/* !checksrc! disable COPYRIGHT all */' > $(HUGE);   \
        echo '#include "tool_hugehelp.h"' >> $(HUGE)
 endif
 
@@ -168,7 +175,7 @@ curl_cfiles_gen += $(HUGE)
 curl_hfiles_gen += tool_hugehelp.h
 CLEANFILES += $(HUGE)
 
-CA_EMBED_CSOURCE = $(builddir)/tool_ca_embed.c
+CA_EMBED_CSOURCE = tool_ca_embed.c
 curl_cfiles_gen += $(CA_EMBED_CSOURCE)
 CLEANFILES += $(CA_EMBED_CSOURCE)
 if CURL_CA_EMBED_SET
@@ -178,7 +185,8 @@ $(CA_EMBED_CSOURCE): $(MK_FILE_EMBED) $(CURL_CA_EMBED)
        $(PERL) $(MK_FILE_EMBED) --var curl_ca_embed < $(CURL_CA_EMBED) > $(CA_EMBED_CSOURCE)
 else
 $(CA_EMBED_CSOURCE):
-       echo 'extern const void *curl_ca_embed; const void *curl_ca_embed;' > $(CA_EMBED_CSOURCE)
+       echo '/* !checksrc! disable COPYRIGHT all */' > $(CA_EMBED_CSOURCE)
+       echo 'extern const void *curl_ca_embed; const void *curl_ca_embed;' >> $(CA_EMBED_CSOURCE)
 endif
 
 CHECKSRC = $(CS_$(V))
@@ -190,10 +198,12 @@ CS_ = $(CS_0)
 checksrc:
        $(CHECKSRC)(@PERL@ $(top_srcdir)/scripts/checksrc.pl -D$(srcdir) $(CURL_CFILES) $(CURL_HFILES))
 
+if NOT_CURL_CI
 if DEBUGBUILD
 # for debug builds, we scan the sources on all regular make invokes
 all-local: checksrc
 endif
+endif
 
 # disable the tests that are mostly causing false positives
 TIDYFLAGS := -checks=-clang-analyzer-security.insecureAPI.strcpy,-clang-analyzer-optin.performance.Padding,-clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling -quiet
index e4dbe35de1acdcf1283b4e3fbae1ca6f606a013a..9daa519e89ffba2f7ab059ec1cb436aab06e0a28 100755 (executable)
@@ -35,6 +35,7 @@ print <<HEAD
 /*
  * NEVER EVER edit this manually, fix the mk-file-embed.pl script instead!
  */
+/* !checksrc! disable COPYRIGHT all */
 #ifndef CURL_DECLARED_${varname_upper}
 #define CURL_DECLARED_${varname_upper}
 extern const unsigned char ${varname}[];
index bae747f1d41043431eb5819d6ef4f5d2f527d3e3..d1acb95da1e67dc2bd9e46183377bd16b8c56e26 100644 (file)
@@ -70,3 +70,7 @@ CS_ = $(CS_0)
 
 checksrc:
        $(CHECKSRC)(@PERL@ $(top_srcdir)/scripts/checksrc.pl -D$(srcdir) $(srcdir)/*.c)
+
+if NOT_CURL_CI
+all-local: checksrc
+endif
index 497493cc2cd2bb78cdbb73e0a1caa73ff0e3975f..07980f85261ef4f5b9c3bbe37a0f6adb6889c4c1 100644 (file)
@@ -127,13 +127,12 @@ CS_1 =
 CS_ = $(CS_0)
 
 # ignore generated C files since they play by slightly different rules!
-checksrc:
+checksrc: lib1521.c
        $(CHECKSRC)(@PERL@ $(top_srcdir)/scripts/checksrc.pl -D$(srcdir) \
          -W$(srcdir)/libtest_bundle.c \
          $(srcdir)/*.[ch])
 
-if DEBUGBUILD
-# for debug builds, we scan the sources on all regular make invokes
+if NOT_CURL_CI
 all-local: checksrc
 endif
 
index 6ca51392c7518529a0afc3ada8032b6103c04092..982de7063506d2499f66a36c1b26e92acba5780f 100755 (executable)
@@ -39,6 +39,7 @@ open my $fh, "<", "$src_dir/Makefile.inc" or die "Cannot open '$src_dir/Makefile
 print <<HEADER
 /* !checksrc! disable COPYRIGHT all */
 /* !checksrc! disable INCLUDEDUP all */
+/* !checksrc! disable UNUSEDIGNORE all */
 
 #define CURLTESTS_BUNDLED
 #define CURLTESTS_BUNDLED_TEST_H
index 3b3e6a80e01f0570f772380a92e6bb4c032f7f88..8298119e881480c3a03168d73e7bb96fe0720aa5 100644 (file)
@@ -95,7 +95,6 @@ checksrc:
          -W$(srcdir)/server_bundle.c \
          $(srcdir)/*.[ch])
 
-if DEBUGBUILD
-# for debug builds, we scan the sources on all regular make invokes
+if NOT_CURL_CI
 all-local: checksrc
 endif
index 92e160b2d3a3b52ddc9c940505553bc26b581adc..01d27673c449d90ce01fd1a46a0db6d86d353192 100644 (file)
@@ -90,5 +90,9 @@ checksrc:
          -W$(srcdir)/tool_bundle.c \
          $(srcdir)/*.[ch])
 
+if NOT_CURL_CI
+all-local: checksrc
+endif
+
 clean-local:
        rm -f $(BUNDLE)
index 9cf63aca139001fe31d691a95eaf61ab3dac1e3e..331d0a9004800d081eefdd4efe23e68dc66be6b6 100644 (file)
@@ -88,5 +88,9 @@ checksrc:
          -W$(srcdir)/unit_bundle.c \
          $(srcdir)/*.[ch])
 
+if NOT_CURL_CI
+all-local: checksrc
+endif
+
 clean-local:
        rm -f $(BUNDLE)