9959 2016-07-15 21:12:16 +0000 patchtest-oe: the check for 'signed-off-by' format is failing on unicode names 2017-12-06 17:03:53 +0000 1 1 6 Yocto Project Subprojects Patchwork/Patchtest Patchtest 2.3 x86 Multiple RESOLVED FIXED Medium+ major 2.2 M3 1 leonardo.sandoval.gonzalez daniela.plascencia sgw --- --- No (bug/feature does not impact docs) oldest_to_newest 64069 0 leonardo.sandoval.gonzalez 2016-07-15 21:12:16 +0000 Current parser (pyparsing) is not parsing correctly those signed-off-by patches that contain unicode chars (like Aníbal Limón), thus giving false positives: > git pw mbox 337 | patchtest - --test-dir ~/repos/pt-oe/tests --no-apply --raw ---------------------------------------------------------------------- Ran 4 tests in 6.249s OK [["ID", "mbox.signed_off_by.test_signed_off_by.SignedOffBy.test_signed_off_by_format"], ["line", "Signed-off-by: An\u00edbal Lim\u00f3n <anibal.limon@linux.intel.com>"], ["column", 18]] [["ID", "mbox.subject.test_subject.Subject.test_subject_length"], ["Subject", "utils/qemurunner.py: QemuRunner.start() add support for specify extra kernel cmdline"], ["Length", "Current length 84 Max length 80"]] where .git/config contains [patchwork "default"] url = http://patchwork.openembedded.org project = oe-core 65646 1 leonardo.sandoval.gonzalez 2016-08-31 14:58:56 +0000 Instead of using pyparsing, standard (regular expression) python module can be used. 65922 2 daniela.plascencia 2016-09-08 14:01:10 +0000 Using pyparsing regular expressions to fix this bug.