Bug 11305

Summary:	patchwork: Improve Series naming when no cover letter is provided
Product:	[Yocto Project Subprojects] Patchwork/Patchtest	Reporter:	Jose Lamego <jose.a.lamego>
Component:	Patchwork	Assignee:	Jose Lamego <jose.a.lamego>
Status:	RESOLVED FIXED	QA Contact:
Severity:	enhancement
Priority:	Medium+	CC:	leonardo.sandoval.gonzalez
Version:	2.4
Target Milestone:	2.4 M1
Hardware:	x86
OS:	Multiple
Whiteboard:
OS type for building Yocto:	---	Type of Regression:	---
Verified:		Documentation change:	No (bug/feature does not impact docs)

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