summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRoss Burton <ross.burton@intel.com>2018-05-11 17:08:09 +0100
committerRichard Purdie <richard.purdie@linuxfoundation.org>2018-05-15 10:56:51 +0100
commit076d8fcdd210d8b8f76657f2c464955cb4313614 (patch)
tree49a6bbeabb0aa4532cdf188a8c5c7c3e2b47187b
parentdb4965438bc88c32eda4a0b86bd731b796d27502 (diff)
downloadpoky-076d8fcdd210d8b8f76657f2c464955cb4313614.tar.gz
poky-076d8fcdd210d8b8f76657f2c464955cb4313614.tar.bz2
poky-076d8fcdd210d8b8f76657f2c464955cb4313614.zip
patchreview: don't disable malformed SoB check
We cleaned up the metadata so this can be enabled again. (From OE-Core rev: 9611485bba03ef77ff31121e3b1da7cd57990c3e) Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rwxr-xr-xscripts/contrib/patchreview.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/scripts/contrib/patchreview.py b/scripts/contrib/patchreview.py
index 4e3e73c7a8..1086c95f67 100755
--- a/scripts/contrib/patchreview.py
+++ b/scripts/contrib/patchreview.py
@@ -132,8 +132,8 @@ def analyse(results, want_blame=False, verbose=True):
need_blame = True
if verbose:
print("Missing Signed-off-by tag (%s)" % patch)
- # TODO: disable this for now as too much fails
- if False and r.malformed_sob:
+
+ if r.malformed_sob:
need_blame = True
if verbose:
print("Malformed Signed-off-by '%s' (%s)" % (r.malformed_sob, patch))