Bug 11711

Summary: check proper prefix on subject
Product: [Yocto Project Subprojects] Patchwork/Patchtest Reporter: Leonardo Sandoval Gonzalez <leonardo.sandoval.gonzalez>
Component: Patchtest TestsuiteAssignee: Daniela Plascencia <daniela.plascencia>
Status: RESOLVED INVALID QA Contact:
Severity: normal    
Priority: Medium CC: bluelightning, daniela.plascencia, ross.burton
Version: 2.4   
Target Milestone: 2.5   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)

Description Leonardo Sandoval Gonzalez 2017-06-23 15:20:02 UTC
The shortlog's prefix indicates several things besides indicating that it is a patch: revision, targeted branch, patch number and total patches. This info needs to be present in a standard format, thus making it easier to parse it. Create a tests to check for the latter.
Comment 1 Leonardo Sandoval Gonzalez 2017-06-29 14:53:49 UTC
(In reply to comment #0)
> The shortlog's prefix indicates several things besides indicating that it is
> a patch: revision, targeted branch, patch number and total patches. This
> info needs to be present in a standard format, thus making it easier to
> parse it. Create a tests to check for the latter.

To be more precise, the only check required is the targeted branch format if present.

On the mailing lists. At least for pyro/morty, these are the flavors seen

    48:./7014.1.mbox:5:Subject: [1/3,pyro] piglit: depend on virtual/egl
     49:./7014.1.mbox:51:Subject: [2/3,pyro] piglit: add patch for lack of gbm_bo_map
     51:./7014.1.mbox:143:Subject: [3/3,pyro] piglit: add patches for unbuildable surfaceless Mesa test
    119:./7008.3.mbox:5:Subject: [pyro] bitbake.conf: Add sdl-config to HOSTTOOLS if using host SDL
    121:./6991.4.mbox:5:Subject: [pyro] archiver: Escape recipe name in regex
    190:./7118.1.mbox:5:Subject: [1/3,v2,pyro] piglit: depend on virtual/egl
    191:./7118.1.mbox:53:Subject: [2/3,v2,pyro] piglit: add patch for lack of gbm_bo_map
    210:./7129.1.mbox:5:Subject: [master, pyro] image-vm: Avoid use of fold,
    335:./6060.2.mbox:5:Subject: [master,pyro] bitbake.conf: Add whoami to HOSTTOOLS
    434:./7165.2.mbox:5:Subject: [pyro] busybox: add backported patch to support iproute 'scope'
    437:./7166.2.mbox:5:Subject: [pyro] openssh: allow to override OpenSSL HostKeys when
    520:./6623.3.mbox:5:Subject: [pyro] automake: Backport perl 5.22 fix
    886:./7349.1.mbox:5:Subject: [pyro] autogen-native: Set POSIX_SHELL to /bin/sh


     36:./7010.1.mbox:5:Subject: [morty,1/1] perf: add perf-feature for systemtap
     92:./6194.2.mbox:5:Subject: [morty] eudev: set LGPL-2.1+ for libudev package
    120:./6991.3.mbox:5:Subject: [morty] archiver: Escape recipe name in regex
    199:./6623.2.mbox:5:Subject: [morty] automake: Backport perl 5.22 fix
    202:./7122.1.mbox:5:Subject: [morty] gcc-6.2: backport fix of check for empty string in ubsan.c
    204:./7124.1.mbox:5:Subject: [morty] elfutils: fix building elfutils-native with GCC7
    264:./7141.1.mbox:5:Subject: [morty] linux-yocto/4.8: update to 4.8.24
    799:./6194.3.mbox:5:Subject: [morty] eudev: set LGPL-2.1+ for libudev package
    942:./7398.1.mbox:5:Subject: (morty) automake: fix Perl regex
    945:./7399.1.mbox:5:Subject: (morty) gcc: fix ubsan empty string check
    947:./7400.1.mbox:5:Subject: (morty) elfutils: fix fallthrough warnings and printf overflow
    956:./6627.6.mbox:5:Subject: [v2,morty] rpm: Fix the Bug of SRPM String error
    961:./7405.1.mbox:5:Subject: [v2] (morty) Arch Linux build fixes
    978:./7416.1.mbox:5:Subject: [v2] (morty) rpm: Fix the Bug of SRPM String error
   1005:./7416.2.mbox:5:Subject: [v3] (morty) rpm: Fix the Bug of SRPM String error

at least we should document a standard way and check closing brackets instead of parenthesis.
Comment 2 Leonardo Sandoval Gonzalez 2017-11-03 18:37:34 UTC
There is no definite way to provide the branch name so patchtest cannot force a certain format. Setting this bug as invalid.