Re: Maintaining patchtest
Paul,
toggle quoted message
Show quoted text
Thanks for stepping up as the Maintainer. - Armin On 1/8/20 4:48 AM, Paul Barker wrote:
Hi all, After discussion on the technical call yesterday I'm looking at becoming the maintainer of patchtest (http://git.yoctoproject.org/cgit/cgit.cgi/patchtest/) and patchtest-oe (http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe/) for Yocto Project. Aside from a few minor issues I think patchtest is working pretty well at the moment so I don't want to rock the boat too much. However I do want to extend patchtest in a couple of ways: * Support testing commits on a branch as well as patches in a directory. This will allow integration with merge request workflows as well as patches-by-email workflows. * Support testing patches to other layers as well as oe-core. This will require splitting the test cases into a "common" set which apply to all layers (e.g. "Signed-off-by" must be present in a patch) and an "oe-core specific" set (e.g. bugzilla reference validation which may not be relevant for third-party layers). Other layers will then be able to include their own test cases. * Make it easier to run patchtest locally before submitting patches to the list. I also want to simplify future maintenance of patchtest in a couple of ways: * Move existing test cases from patchtest-oe into oe-core itself. Archive off the old patchtest-oe repository. * Consider if patchtest itself can be moved into oe-core as a script. This is desirable but there would be a few issues to overcome, I need to look into how patchtest is actually invoked by patchwork on the server side first. * If patchtest is to remain a separate repository then I'll move development to GitLab with git.yoctoproject.org becoming a mirror. This will allow automation of a few things like the deployment of new patchtest versions to the server after committing to the master branch. I've also still got my eye on other ways we can support merge request workflows for layers that want to use this but December got busy. I'll give this a week for people to reply with feedback and then submit patches to update the maintainers files in patchtest & patchtest-oe. Thanks, Paul |
|