11755 2017-07-04 22:52:42 +0000 patchtest-oe: test checking changes in lic_files_chcksum not failing when lacking the explanation in commit message 2017-10-20 20:45:08 +0000 1 1 6 Yocto Project Subprojects Patchwork/Patchtest Patchtest Testsuite unspecified x86 Multiple RESOLVED FIXED Medium normal 2.5 M1 1 daniela.plascencia daniela.plascencia leonardo.sandoval.gonzalez --- --- No (bug/feature does not impact docs) oldest_to_newest 74756 0 daniela.plascencia 2017-07-04 22:52:42 +0000 The current test already has a check that should send a failure message when the LIC_FILES_CHKSUM is modified and the explanation as to why is not present on the commit message. For some reason it is not detecting the patches lacking the explanation and not failing properly. The usage of Tinfoil may solve this issue. 75108 1 leonardo.sandoval.gonzalez 2017-07-18 13:57:43 +0000 This issue may be resolved with current master-next branches which will land to master shortly. 76702 2 daniela.plascencia 2017-09-08 20:47:52 +0000 (In reply to comment #1) > This issue may be resolved with current master-next branches which will land > to master shortly. This issue is not present anymore considering the following: a) Both framework and test suite are on master-next b) Latest change on base.py: find_pn() returns PN instead of FN (PATH) c) All metadata test cases use parse_recipe() with PN as input instead of FN b and c are not in master-next, but certainly helped solving runtime errors and might be merged soon. 76703 3 daniela.plascencia 2017-09-08 20:50:32 +0000 Changing status "IN PROGRESS REVIEW". 77751 4 daniela.plascencia 2017-10-20 20:45:08 +0000 Latest changes on master solve this issue. http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe/log/