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.
(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.
There is no definite way to provide the branch name so patchtest cannot force a certain format. Setting this bug as invalid.