Check that patch has dependencies update

To disable the new check, one can add NO_DEPS_UPDATE tag to the
commit message.

Signed-off-by: Sergey Bronnikov <sergeyb@tarantool.org>
Reviewed-by: Vladimir Davydov <vdavydov@tarantool.org>

Author: ligurio

checkpatch.pl

@@ -497,7 +497,8 @@ sub hash_show_words {
 our $custom_tags = qr{(?x:
 	NO_DOC|
 	NO_TEST|
-	NO_CHANGELOG
+	NO_CHANGELOG|
+	NO_DEPS_UPDATE
 )};
 
 our $github_ref = qr{(https?:\/\/)?(www\.)?github\.com\b([-a-zA-Z0-9()@:%_\+.~\#?&\/=]*)};
@@ -2156,6 +2157,7 @@ sub process {
 	my $new_file = 0;
 	my $has_doc = 0;
 	my $has_test = 0;
+	my $has_deps = 0;
 	my $is_test = 0;
 
 	# Pre-scan the patch sanitizing the lines.
@@ -2747,7 +2749,7 @@ sub process {
 			}
 		}
 
-		if ($in_commit_log && $line =~ /^($custom_tags)=/) {
+		if ($in_commit_log && $line =~ /^($custom_tags)=?/) {
 			$commit_log_tags{$1} = 1;
 			if ($has_doc) {
 				ERROR("TAG_IN_DOC",
@@ -2767,6 +2769,9 @@ sub process {
 		if ($realfile =~ /^(?:static-build\/)?test\/.*\//) {
 			$has_test = 1;
 		}
+		if ($realfile =~ /^(?:third_party|cmake)\//) {
+			$has_deps = 1;
+		}
 
 		$is_test = ($realfile =~ /^(?:test|perf)\//);
 
@@ -5293,6 +5298,11 @@ sub process {
 			ERROR("NO_TEST",
 			      "Please add test or NO_TEST=<reason> tag\n");
 		}
+		if ($has_deps && !exists($commit_log_tags{'NO_DEPS_UPDATE'}) &&
+			!($filename !~ /^extra\/dependencies\.yaml$/)) {
+			ERROR("NO_DEPS_UPDATE",
+			      "Please update dependencies or add NO_DEPS_UPDATE tag\n");
+		}
 	}
 
 	print report_dump();

doc/checkpatch.rst

@@ -766,6 +766,12 @@ Others
     If the patch doesn't need a test (e.g. it fixes a CI issue), please add
     NO_TEST=<reason> to the commit log.
 
+  **NO_DEPS_UPDATE**
+    The patch has dependencies update.  Please update dependencies versions or
+    add a new entry to the file with dependencies extra/dependencies.yaml.
+    If the patch doesn't update dependencies (e.g. it fixes a typo), please add
+    NO_DEPS_UPDATE to the commit log.
+
   **NOT_UNIFIED_DIFF**
     The patch file does not appear to be in unified-diff format.  Please
     regenerate the patch file before sending it to the maintainer.