Bug 10688 - Add a test looking for common paths that indicate patch is for the wrong layer
Summary: Add a test looking for common paths that indicate patch is for the wrong layer
Status: RESOLVED FIXED
Alias: None
Product: Patchwork/Patchtest
Classification: Yocto Project Subprojects
Component: Patchtest Testsuite (show other bugs)
Version: unspecified
Hardware: x86 Multiple
: Medium+ enhancement
Target Milestone: 2.3 M3
Assignee: Leonardo Sandoval Gonzalez
QA Contact:
URL:
Whiteboard:
: 9944 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-11-16 21:45 UTC by Paul Eggleton
Modified: 2017-12-06 17:16 UTC (History)
1 user (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Eggleton 2016-11-16 21:45:50 UTC
If a patch against meta-oe is sent incorrectly to the OE-Core mailing list and it is only adding files, it will apply cleanly on top of OE-Core since it doesn't matter that there isn't already an existing meta-oe subdirectory. It would be useful if we had a test to catch these kinds of patches, but if the email subject doesn't contain the layer name (which is sometimes omitted by the submitter) then the only way it could work would be if it looked for subdirectories from common layers such as "meta-oe". The suggested fix should specify to resend to the correct mailing list, specifying the actual mailing list based on subdirectory.

We'd want an easy way for these tests to be reconfigured of course so that the same set of tests can be easily applied e.g. for meta-oe when the time comes to do that - there we would of course be wanting to check for patches against a "meta" subdirectory. Perhaps we'd need to have a config option somewhere to accomplish this.
Comment 1 Leonardo Sandoval Gonzalez 2017-02-08 20:44:58 UTC
*** Bug 9944 has been marked as a duplicate of this bug. ***
Comment 2 Leonardo Sandoval Gonzalez 2017-02-09 20:41:24 UTC
Patch sent to the ML:

https://lists.yoctoproject.org/pipermail/yocto/2017-February/034467.html