[OE-core] [PATCH] patchreview: don't disable malformed SoB check

[Ross Burton]

We cleaned up the metadata so this can be enabled again.

Signed-off-by: Ross Burton 
---
 scripts/contrib/patchreview.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/contrib/patchreview.py b/scripts/contrib/patchreview.py
index 4e3e73c7a8b..1086c95f67f 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))
-- 
2.11.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] patchreview: don't disable malformed SoB check

[Ross Burton]

We cleaned up the metadata so this can be enabled again.

Signed-off-by: Ross Burton 
---
 scripts/contrib/patchreview.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/contrib/patchreview.py b/scripts/contrib/patchreview.py
index 4e3e73c7a8b..1086c95f67f 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))
-- 
2.11.0

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core