Re: ✗ patchtest: failure for Allow PR Service and hash equiv together

Mark Hatle
2020-08-25T01:17:21Z

On 8/24/20 6:32 PM, Patchwork wrote:
> == Series Details ==
>
> Series: Allow PR Service and hash equiv together
> Revision: 1
> URL : https://patchwork.openembedded.org/series/25758/
> State : failure
>
> == Summary ==
>
>
> Thank you for submitting this patch series to OpenEmbedded Core. This is
> an automated response. Several tests have been executed on the proposed
> series by patchtest resulting in the following failures:
>
>
>
> * Patch [master,2/5] hash equivalency and pr service
> Issue Shortlog does not follow expected format [test_shortlog_format]
> Suggested fix Commit shortlog (first line of commit message) should follow the format "<target>: "

In this case, I think what I did was acceptable. If reviewers disagree I'll
change the format.

> * Issue Errors in your Python code were encountered [test_pylint]
> Suggested fix Correct the lines introduced by your patch
> Output Please, fix the listed issues:
> meta/lib/oe/packagedata.py does not exist

Now this one is _confusing_, it absolutely exists:

https://git.openembedded.org/openembedded-core/tree/meta/lib/oe/packagedata.py

Is the link engine broken?

> If you believe any of these test results are incorrect, please reply to the
> mailing list (openembedded-core@lists.openembedded.org) raising your concerns.

DONE!

--Mark

> Otherwise we would appreciate you correcting the issues and submitting a new
> version of the patchset if applicable. Please ensure you add/increment the
> version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
> [PATCH v3] -> ...).
>
> ---
> Guidelines: https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
> Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
> Test suite: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe
>