fixed #14000 - deprecated the usage of rules [skip ci]

Author: firewave

Makefile

@@ -150,6 +150,7 @@ ifeq ($(HAVE_RULES),yes)
     else
         LIBS=$(shell $(PCRE_CONFIG) --libs)
     endif
+    $(warning The usage of rules has been deprecated and will be removed in a future Cppcheck version.)
 else ifneq ($(HAVE_RULES),)
     $(error invalid HAVE_RULES value '$(HAVE_RULES)')
 endif

cli/cmdlineparser.cpp

@@ -59,6 +59,10 @@
 #ifdef HAVE_RULES
 // xml is used for rules
 #include "xml.h"
+
+#ifdef _MSC_VER
+#pragma message("The usage of rules is deprecated and will be removed in a future Cppcheck version. Please use addons instead.")
+#endif
 #endif
 
 static bool addFilesToList(const std::string& fileList, std::vector<std::string>& pathNames)
@@ -1258,6 +1262,8 @@ CmdLineParser::Result CmdLineParser::parseFromArgs(int argc, const char* const a
                 return Result::Fail;
             }
 
+            mLogger.printMessage("'--rule' has been deprecated and will be removed in a future Cppcheck version. Please use an addon instead.");
+
             mSettings.rules.emplace_back(std::move(rule));
 #else
             mLogger.printError("Option --rule cannot be used as Cppcheck has not been built with rules support.");
@@ -1340,6 +1346,8 @@ CmdLineParser::Result CmdLineParser::parseFromArgs(int argc, const char* const a
                         return Result::Fail;
                     }
 
+                    mLogger.printMessage("'--rule-file' has been deprecated and will be removed in a future Cppcheck version. Please use an addon instead.");
+
                     mSettings.rules.emplace_back(std::move(rule));
                 }
             } else {

cmake/options.cmake

@@ -80,6 +80,9 @@ if(NOT BUILD_GUI)
 endif()
 
 option(HAVE_RULES           "Usage of rules (needs PCRE library and headers)"               OFF)
+if(HAVE_RULES)
+    message(WARNING "The usage of rules has been deprecated and will be removed in a future Cppcheck version. Please use addons instead.")
+endif()
 option(USE_BUNDLED_TINYXML2 "Usage of bundled TinyXML2 library"                             ON)
 if(BUILD_CORE_DLL AND NOT USE_BUNDLED_TINYXML2)
     message(FATAL_ERROR "Cannot use external TinyXML2 library when building lib as DLL")

releasenotes.txt

@@ -16,6 +16,7 @@ Deprecations:
 - Support for building with Qt 5 will be removed in Cppcheck 2.19.
 - The platform 'unix32-unsigned' has been deprecated and will be removed in Cppcheck 2.19. Please use '--platform=unix32 --funsigned-char' instead.
 - The platform 'unix64-unsigned' has been deprecated and will be removed in Cppcheck 2.19. Please use '--platform=unix64 --funsigned-char' instead.
+- The usage of rules has been deprecated and will be removed in a future Cppcheck version. Please use addons instead.
 -
 
 Other:

tools/dmake/dmake.cpp

@@ -746,6 +746,7 @@ int main(int argc, char **argv)
          << "    else\n"
          << "        LIBS=$(shell $(PCRE_CONFIG) --libs)\n"
          << "    endif\n"
+         << "    $(warning The usage of rules has been deprecated and will be removed in a future Cppcheck version. Please use addons instead.)\n"
          << "else ifneq ($(HAVE_RULES),)\n"
          << "    $(error invalid HAVE_RULES value '$(HAVE_RULES)')\n"
          << "endif\n\n";