<?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>11780</bug_id>
          
          <creation_ts>2017-07-11 09:46:12 +0000</creation_ts>
          <short_desc>Commit message failures should link to commit patch guidelines</short_desc>
          <delta_ts>2017-10-20 14:49:33 +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</component>
          <version>unspecified</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>2.5 M1</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Joshua Lock">joshuagloe</reporter>
          <assigned_to name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</assigned_to>
          <cc>bluelightning</cc>
    
    <cc>denis</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>74918</commentid>
    <comment_count>0</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2017-07-11 09:46:12 +0000</bug_when>
    <thetext>Patchtest should help users to resolve any reported issues. 

test_shortlog_format, and other tests of the commit message, should include a link to the &quot;Commit Patch Message Guidelines&quot; on the OE wiki to help the submitter rectify any errors

https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>74929</commentid>
    <comment_count>1</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2017-07-11 15:44:49 +0000</bug_when>
    <thetext>I already created a patch [1] but not yet reflected on emails (it is part of master-next to be merged at master &lt; M2). The approach was to append the URL in the footnote area, meaning that all emails coming from pw on behalf of patchest will have the suggested URL. If you think it is better to include depending on the test case, let me know.

[1] http://git.yoctoproject.org/cgit/cgit.cgi/patchtest/commit/?h=master-next&amp;id=cff83280400e4cd8ac4444319e188f9e1b142af9</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77464</commentid>
    <comment_count>2</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2017-10-09 16:43:04 +0000</bug_when>
    <thetext>This is already implemented but cannot be resolved because change is still in master-next, a branch pending to be merged with master.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77739</commentid>
    <comment_count>3</comment_count>
    <who name="Leonardo Sandoval Gonzalez">leonardo.sandoval.gonzalez</who>
    <bug_when>2017-10-20 14:49:33 +0000</bug_when>
    <thetext>Now in master. patchwork&apos;s email footer now has the following strings:

---
Guidelines:     https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite:     http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>