15390 2024-02-08 09:21:13 +0000 patchtest didn't report that the patch was still authored by AUH 2024-09-03 13:24:10 +0000 1 1 6 Yocto Project Subprojects Patchwork/Patchtest Patchtest 5.1 x86 Multiple RESOLVED FIXED Medium+ normal 5.1 M3 1 michael.opdenacker tgamblin michael.opdenacker randy.macleod --- --- No (bug/feature does not impact docs) oldest_to_newest 98154 0 5010 michael.opdenacker 2024-02-08 09:21:13 +0000 Created attachment 5010 Patch with auh@yoctoproject.org as commit author Greetings I tested this on Poky master with the attached patch I prepared from an e-mail sent by the Auto Upgrade Helper: mike@xps:~/work/yocto/poky$ patchtest --patch ~/tmp/patchtest-case/0003-alsa-ucm-conf-upgrade-1.2.10-1.2.11.patch Testing patch /home/mike/tmp/patchtest-case/0003-alsa-ucm-conf-upgrade-1.2.10-1.2.11.patch Loading cache: 100% | | ETA: --:--:-- Loaded 0 entries from dependency cache. Parsing recipes: 100% |#############################################################################################################################################################################| Time: 0:00:12 Parsing of 912 .bb files complete (0 cached, 912 parsed). 1849 targets, 47 skipped, 0 masked, 0 errors. SKIP: pretest src uri left files: Patch cannot be merged (test_metadata.TestMetadata.pretest_src_uri_left_files) SKIP: pretest pylint: No python related patches, skipping test (test_python_pylint.PyLint.pretest_pylint) ---------------------------------------------------------------------- Ran 2 tests in 13.504s OK PASS: test author valid (test_mbox.TestMbox.test_author_valid) SKIP: test bugzilla entry format: No bug ID found (test_mbox.TestMbox.test_bugzilla_entry_format) PASS: test commit message presence (test_mbox.TestMbox.test_commit_message_presence) PASS: test mbox format (test_mbox.TestMbox.test_mbox_format) PASS: test non-AUH upgrade (test_mbox.TestMbox.test_non_auh_upgrade) SKIP: test series merge on head: Merge test is disabled for now (test_mbox.TestMbox.test_series_merge_on_head) PASS: test shortlog format (test_mbox.TestMbox.test_shortlog_format) PASS: test shortlog length (test_mbox.TestMbox.test_shortlog_length) 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: test target mailing list (test_mbox.TestMbox.test_target_mailing_list) Loading cache: 100% |###############################################################################################################################################################################| Time: 0:00:00 Loaded 1849 entries from dependency cache. PASS: test CVE check ignore (test_metadata.TestMetadata.test_cve_check_ignore) PASS: test lic files chksum modified not mentioned (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) PASS: test max line length (test_metadata.TestMetadata.test_max_line_length) SKIP: test src uri left files: Patch cannot be merged (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 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 pylint: No python related patches, skipping test (test_python_pylint.PyLint.test_pylint) ---------------------------------------------------------------------- Ran 21 tests in 1.090s OK (Don't pay attention to the "FAIL" line, it's a different issue addressed by https://bugzilla.yoctoproject.org/show_bug.cgi?id=15389). Patchtest should have warned me that the commit author was still "auh@yoctoproject.org <auh@yoctoproject.org>" and that I forgot to run "git commit --amend --reset-author". I realized this, but after my first submission! 99504 1 tgamblin 2024-08-06 19:25:27 +0000 I'm not totally sure this is a bad thing - do we want to completely omit mention of the AUH patch origin, or do we just want some confidence that the submitter actually tested a patch that they may have taken from AUH? In the example attached here, the mbox's "From:" line is the submitter, but there's a second "From: <auh_email>" in the body (the bigger issue is that the S-O-B is missing!). I think this should be fine, but maybe not? 99595 2 tgamblin 2024-09-01 00:52:30 +0000 Patch sent: https://lists.openembedded.org/g/openembedded-core/message/203978 99602 3 tgamblin 2024-09-03 13:24:10 +0000 See: https://git.openembedded.org/openembedded-core/commit/?id=557400648b6f4f31176847f8a068d2e199b7793d 5010 2024-02-08 09:21:13 +0000 2024-02-08 09:21:13 +0000 Patch with auh@yoctoproject.org as commit author 0003-alsa-ucm-conf-upgrade-1.2.10-1.2.11.patch text/plain 1465 michael.opdenacker RnJvbSAxZDExZGQwNGNhZmNlNDliMjM3NTEzMWZjMDA0YmEyN2Y1Y2QwMWU2IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBNaWNoYWVsIE9wZGVuYWNrZXIgPG1pY2hhZWwub3BkZW5hY2tl ckBib290bGluLmNvbT4KRGF0ZTogVGh1LCAxIEZlYiAyMDI0IDE4OjAwOjA3ICswMDAwClN1Ympl Y3Q6IFtQQVRDSCAzLzRdIGFsc2EtdWNtLWNvbmY6IHVwZ3JhZGUgMS4yLjEwIC0+IDEuMi4xMQoK RnJvbTogYXVoQHlvY3RvcHJvamVjdC5vcmcgPGF1aEB5b2N0b3Byb2plY3Qub3JnPgoKLSBVcHN0 cmVhbSByZWxlYXNlIHVwZGF0ZQogIFNlZSBodHRwczovL3d3dy5hbHNhLXByb2plY3Qub3JnL3dp a2kvRGV0YWlsZWRfY2hhbmdlc192MS4yLjEwX3YxLjIuMTEKLS0tCiAuLi4vYWxzYS97YWxzYS11 Y20tY29uZl8xLjIuMTAuYmIgPT4gYWxzYS11Y20tY29uZl8xLjIuMTEuYmJ9ICAgfCAyICstCiAx IGZpbGUgY2hhbmdlZCwgMSBpbnNlcnRpb24oKyksIDEgZGVsZXRpb24oLSkKIHJlbmFtZSBtZXRh L3JlY2lwZXMtbXVsdGltZWRpYS9hbHNhL3thbHNhLXVjbS1jb25mXzEuMi4xMC5iYiA9PiBhbHNh LXVjbS1jb25mXzEuMi4xMS5iYn0gKDg5JSkKCmRpZmYgLS1naXQgYS9tZXRhL3JlY2lwZXMtbXVs dGltZWRpYS9hbHNhL2Fsc2EtdWNtLWNvbmZfMS4yLjEwLmJiIGIvbWV0YS9yZWNpcGVzLW11bHRp bWVkaWEvYWxzYS9hbHNhLXVjbS1jb25mXzEuMi4xMS5iYgpzaW1pbGFyaXR5IGluZGV4IDg5JQpy ZW5hbWUgZnJvbSBtZXRhL3JlY2lwZXMtbXVsdGltZWRpYS9hbHNhL2Fsc2EtdWNtLWNvbmZfMS4y LjEwLmJiCnJlbmFtZSB0byBtZXRhL3JlY2lwZXMtbXVsdGltZWRpYS9hbHNhL2Fsc2EtdWNtLWNv bmZfMS4yLjExLmJiCmluZGV4IGY0ZmQyODRkYjguLmNlMWE0MmM5YjggMTAwNjQ0Ci0tLSBhL21l dGEvcmVjaXBlcy1tdWx0aW1lZGlhL2Fsc2EvYWxzYS11Y20tY29uZl8xLjIuMTAuYmIKKysrIGIv bWV0YS9yZWNpcGVzLW11bHRpbWVkaWEvYWxzYS9hbHNhLXVjbS1jb25mXzEuMi4xMS5iYgpAQCAt OCw3ICs4LDcgQEAgTElDRU5TRSA9ICJCU0QtMy1DbGF1c2UiCiBMSUNfRklMRVNfQ0hLU1VNID0g ImZpbGU6Ly9MSUNFTlNFO21kNT0yMGQ3NGQ3NGRiOTc0MTY5NzkwMzM3MmFkMDAxZDNiNCIKIAog U1JDX1VSSSA9ICJodHRwczovL3d3dy5hbHNhLXByb2plY3Qub3JnL2ZpbGVzL3B1Yi9saWIvJHtC UH0udGFyLmJ6MiIKLVNSQ19VUklbc2hhMjU2c3VtXSA9ICI5YzIxZTNmMDFmZjAwYmFhNzU4ZGYx N2U4NjdjZDM2ZTI0ZWJiNDFhNmJlYzQ5NzM3ZTk5MTA1ZTE2ZjJhZTk3IgorU1JDX1VSSVtzaGEy NTZzdW1dID0gIjM4N2MwMWNmMzBlMmExNjc2ZDdiOGY3MmIyNjgxY2YyMTlhYmNhNzBkZDFlYzJh OWUzM2FkZDViZjNmZWFlODEiCiAjIFNvbWV0aGluZyB3ZW50IHdyb25nIGF0IHVwc3RyZWFtIHRh cmJhbGxpbmcKIAogaW5oZXJpdCBhbGxhcmNoCi0tIAoyLjM0LjEKCg==