From 6d4f01fdeac9ac94d20f7637fe3944828de413c4 Mon Sep 17 00:00:00 2001 From: "Richard J. Knight" Date: Tue, 21 Mar 2017 16:13:18 -0500 Subject: Allow URL in commit message -Updated URL check to mark a line as tagged line if it contains both "http" and "Reviewed-on", other URL instances will be marked as untagged lines. Change-Id: I90c12fe1c763fc083e4d7b349adc4fed7c9a5c15 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38257 Reviewed-by: Elizabeth K. Liner Tested-by: Jenkins Server Tested-by: FSP CI Jenkins Tested-by: Jenkins OP Build CI Reviewed-by: Stephen M. Cprek Reviewed-by: Matthew A. Ploetz --- src/build/tools/verify-commit | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/build/tools/verify-commit b/src/build/tools/verify-commit index 146cabcde..6f04dd97d 100755 --- a/src/build/tools/verify-commit +++ b/src/build/tools/verify-commit @@ -341,8 +341,9 @@ sub verifyCommitMsg # Identify if this is a tagged line or a non-tagged line and store # away. - if ( ($line =~ m/^\s*[A-Za-z0-9\-_]+:[^:]/) - && !($line =~ m/http/) ) + if ( (($line =~ m/^\s*[A-Za-z0-9\-_]+:[^:]/) + && !($line =~ m/http/)) || + (($line =~ m/http/) && ($line =~ m/Reviewed-on:/))) { # We allow lines that look like tags in the topic like... # "FOO: Adding support for BAR." -- cgit v1.2.1