fixed #14000 - deprecated the usage of rules

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.");
@@ -1342,6 +1348,8 @@ CmdLineParser::Result CmdLineParser::parseFromArgs(int argc, const char* const a
 
                     mSettings.rules.emplace_back(std::move(rule));
                 }
+
+                mLogger.printMessage("'--rule-file' has been deprecated and will be removed in a future Cppcheck version. Please use an addon instead.");
             } else {
                 mLogger.printError("unable to load rule-file '" + ruleFile + "' (" + tinyxml2::XMLDocument::ErrorIDToName(err) + ").");
                 return Result::Fail;

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:

test/testcmdlineparser.cpp

@@ -2491,6 +2491,7 @@ class TestCmdlineParser : public TestFixture {
         ASSERT_EQUALS(1, settings->rules.size());
         auto it = settings->rules.cbegin();
         ASSERT_EQUALS(".+", it->pattern);
+        ASSERT_EQUALS("cppcheck: '--rule' has been deprecated and will be removed in a future Cppcheck version. Please use an addon instead.\n", logger->str());
     }
 
     void ruleMissingPattern() {
@@ -2547,6 +2548,7 @@ class TestCmdlineParser : public TestFixture {
         ASSERT_EQUALS_ENUM(Severity::warning, it->severity);
         ASSERT_EQUALS("ruleId2", it->id);
         ASSERT_EQUALS("ruleSummary2", it->summary);
+        ASSERT_EQUALS("cppcheck: '--rule-file' has been deprecated and will be removed in a future Cppcheck version. Please use an addon instead.\n", logger->str());
     }
 
     void ruleFileSingle() {
@@ -2570,6 +2572,7 @@ class TestCmdlineParser : public TestFixture {
         ASSERT_EQUALS_ENUM(Severity::error, it->severity);
         ASSERT_EQUALS("ruleId", it->id);
         ASSERT_EQUALS("ruleSummary", it->summary);
+        ASSERT_EQUALS("cppcheck: '--rule-file' has been deprecated and will be removed in a future Cppcheck version. Please use an addon instead.\n", logger->str());
     }
 
     void ruleFileEmpty() {
@@ -2594,12 +2597,14 @@ class TestCmdlineParser : public TestFixture {
         ASSERT_EQUALS("cppcheck: error: unable to load rule-file 'rule.xml' (XML_ERROR_EMPTY_DOCUMENT).\n", logger->str());
     }
 
+    // TODO: bail out instead?
     void ruleFileNoRoot() {
         REDIRECT;
         ScopedFile file("rule.xml", "<?xml version=\"1.0\"?>");
         const char * const argv[] = {"cppcheck", "--rule-file=rule.xml", "file.cpp"};
         ASSERT_EQUALS_ENUM(CmdLineParser::Result::Success, parseFromArgs(argv));
         ASSERT_EQUALS(0, settings->rules.size());
+        ASSERT_EQUALS("cppcheck: '--rule-file' has been deprecated and will be removed in a future Cppcheck version. Please use an addon instead.\n", logger->str());
     }
 
     void ruleFileEmptyElements1() {

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";