Bug 11305 - patchwork: Improve Series naming when no cover letter is provided
Summary: patchwork: Improve Series naming when no cover letter is provided
Status: RESOLVED FIXED
Alias: None
Product: Patchwork/Patchtest
Classification: Yocto Project Subprojects
Component: Patchwork (show other bugs)
Version: 2.4
Hardware: x86 Multiple
: Medium+ enhancement
Target Milestone: 2.4 M1
Assignee: Jose Lamego
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2017-04-06 20:41 UTC by Jose Lamego
Modified: 2017-12-06 17:27 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 Jose Lamego 2017-04-06 20:41:37 UTC
Patch Series created without a cover letter are named using either the first 30 characters in patch #1 plus the remaining patches number, or the patch name for
one-patch-series. However, if for any reason the Subject in patch #1 includes the patch number/order as prefix (for example: [1/2]), this is inappropriately included in the Series name.
A better naming should remove the number/order prefix from the Series name.
Comment 1 Jose Lamego 2017-04-07 15:08:26 UTC
Patch sent to the mailing list can be viewed at:
https://lists.yoctoproject.org/pipermail/yocto/2017-April/035549.html
Comment 2 Jose Lamego 2017-04-17 19:30:49 UTC
Patch v2 sent to the mailing list can be viewed in:
https://lists.yoctoproject.org/pipermail/yocto/2017-April/035696.html

This change makes all of the prefixes that may be present at the first patch subject to be dropped for the series naming.
Comment 3 Jose Lamego 2017-05-02 21:57:43 UTC
I submitted a version 3 that omits the number/order prefix when naming a series, without changing any other existing prefix, since this may be needed to identify the patch as related to a git branch other than master (as required in bug 10715). This patch can be viewed at:
https://lists.yoctoproject.org/pipermail/yocto/2017-May/035933.html
Comment 4 Jose Lamego 2017-05-04 18:55:57 UTC
Another patch version (v5) submitted to use group naming instead of referencing by integer and to fix the argument formatting in re.sub function can be viewed at:
https://lists.yoctoproject.org/pipermail/yocto/2017-May/035956.html
Comment 5 Jose Lamego 2017-05-24 16:38:38 UTC
This fix is deployed in production:
9b3b3b49188ee7954b653f295a66498d4b8ac88a