diff options
| author | Richard J. Knight <rjknight@us.ibm.com> | 2017-03-21 16:13:18 -0500 |
|---|---|---|
| committer | Matthew A. Ploetz <maploetz@us.ibm.com> | 2017-03-22 11:09:12 -0400 |
| commit | 6d4f01fdeac9ac94d20f7637fe3944828de413c4 (patch) | |
| tree | fd0891e66c38e1b55c64d1c3a68f6d65b82316ab /src/build/tools | |
| parent | a89ef1040290c7f25fa17f393acdba9a73b67e0a (diff) | |
| download | talos-hostboot-6d4f01fdeac9ac94d20f7637fe3944828de413c4.tar.gz talos-hostboot-6d4f01fdeac9ac94d20f7637fe3944828de413c4.zip | |
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 <eliner@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com>
Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com>
Diffstat (limited to 'src/build/tools')
| -rwxr-xr-x | src/build/tools/verify-commit | 5 |
1 files 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." |
