Ah.. it requires my SoB when the patch is from someone else? That could explain the situation.
Still, it reported an error to patch 1/5 while the issue is in 2/5. Will check what we can improve in the tool... Kind regards, Marta On Wed, Jul 24, 2024 at 1:28 AM Alexandre Belloni < [email protected]> wrote: > On 23/07/2024 17:41:40+0200, Marta Rybczynska wrote: > > This seems to be a false positive. Possibly the path 2/5 has an issue. > I'm > > looking into the source code of patchtest to verify. > > It is not, 2/5 is missing your SoB > > > > > Regards, > > Marta > > > > On Tue, Jul 23, 2024 at 2:12 PM <[email protected]> > > wrote: > > > > > Thank you for your submission. Patchtest identified one > > > or more issues with the patch. Please see the log below for > > > more information: > > > > > > --- > > > Testing patch > > > > /home/patchtest/share/mboxes/v2-1-5-cve-check-annotate-CVEs-during-analysis.patch > > > > > > FAIL: test Signed-off-by presence: Mbox is missing Signed-off-by. Add > it > > > manually or with "git commit --amend -s" > > > (test_mbox.TestMbox.test_signed_off_by_presence) > > > > > > PASS: pretest pylint (test_python_pylint.PyLint.pretest_pylint) > > > PASS: test author valid (test_mbox.TestMbox.test_author_valid) > > > PASS: test commit message presence > > > (test_mbox.TestMbox.test_commit_message_presence) > > > PASS: test max line length > > > (test_metadata.TestMetadata.test_max_line_length) > > > PASS: test mbox format (test_mbox.TestMbox.test_mbox_format) > > > PASS: test non-AUH upgrade (test_mbox.TestMbox.test_non_auh_upgrade) > > > PASS: test pylint (test_python_pylint.PyLint.test_pylint) > > > PASS: test shortlog format (test_mbox.TestMbox.test_shortlog_format) > > > PASS: test shortlog length (test_mbox.TestMbox.test_shortlog_length) > > > > > > SKIP: pretest src uri left files: No modified recipes, skipping pretest > > > (test_metadata.TestMetadata.pretest_src_uri_left_files) > > > SKIP: test CVE check ignore: No modified recipes or older target > branch, > > > skipping test (test_metadata.TestMetadata.test_cve_check_ignore) > > > SKIP: test CVE tag format: No new CVE patches introduced > > > (test_patch.TestPatch.test_cve_tag_format) > > > SKIP: test Signed-off-by presence: No new CVE patches introduced > > > (test_patch.TestPatch.test_signed_off_by_presence) > > > SKIP: test Upstream-Status presence: No new CVE patches introduced > > > (test_patch.TestPatch.test_upstream_status_presence_format) > > > SKIP: test bugzilla entry format: No bug ID found > > > (test_mbox.TestMbox.test_bugzilla_entry_format) > > > SKIP: test lic files chksum modified not mentioned: No modified > recipes, > > > skipping test > > > > (test_metadata.TestMetadata.test_lic_files_chksum_modified_not_mentioned) > > > SKIP: test lic files chksum presence: No added recipes, skipping test > > > (test_metadata.TestMetadata.test_lic_files_chksum_presence) > > > SKIP: test license presence: No added recipes, skipping test > > > (test_metadata.TestMetadata.test_license_presence) > > > SKIP: test series merge on head: Merge test is disabled for now > > > (test_mbox.TestMbox.test_series_merge_on_head) > > > SKIP: test src uri left files: No modified recipes, skipping pretest > > > (test_metadata.TestMetadata.test_src_uri_left_files) > > > SKIP: test summary presence: No added recipes, skipping test > > > (test_metadata.TestMetadata.test_summary_presence) > > > SKIP: test target mailing list: Series merged, no reason to check other > > > mailing lists (test_mbox.TestMbox.test_target_mailing_list) > > > > > > --- > > > > > > Please address the issues identified and > > > submit a new revision of the patch, or alternatively, reply to this > > > email with an explanation of why the patch should be accepted. If you > > > believe these results are due to an error in patchtest, please submit a > > > bug at https://bugzilla.yoctoproject.org/ (use the 'Patchtest' > category > > > under 'Yocto Project Subprojects'). For more information on specific > > > failures, see: https://wiki.yoctoproject.org/wiki/Patchtest. Thank > > > you! > > > > > > > > > > > > > -- > Alexandre Belloni, co-owner and COO, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#202423): https://lists.openembedded.org/g/openembedded-core/message/202423 Mute This Topic: https://lists.openembedded.org/mt/107503127/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
