<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>11711</bug_id>
          
          <creation_ts>2017-06-23 15:20:02 +0000</creation_ts>
          <short_desc>check proper prefix on subject</short_desc>
          <delta_ts>2017-11-03 18:37:34 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>6</classification_id>
          <classification>Yocto Project Subprojects</classification>
          <product>Patchwork/Patchtest</product>
          <component>Patchtest Testsuite</component>
          <version>2.4</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>INVALID</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>2.5</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</reporter>
          <assigned_to name="Daniela Plascencia">daniela.plascencia</assigned_to>
          <cc>bluelightning</cc>
    
    <cc>daniela.plascencia</cc>
    
    <cc>ross.burton</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>74514</commentid>
    <comment_count>0</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2017-06-23 15:20:02 +0000</bug_when>
    <thetext>The shortlog&apos;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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>74640</commentid>
    <comment_count>1</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2017-06-29 14:53:49 +0000</bug_when>
    <thetext>(In reply to comment #0)
&gt; The shortlog&apos;s prefix indicates several things besides indicating that it is
&gt; a patch: revision, targeted branch, patch number and total patches. This
&gt; info needs to be present in a standard format, thus making it easier to
&gt; 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 &apos;scope&apos;
    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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77954</commentid>
    <comment_count>2</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2017-11-03 18:37:34 +0000</bug_when>
    <thetext>There is no definite way to provide the branch name so patchtest cannot force a certain format. Setting this bug as invalid.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>