<?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>11294</bug_id>
          
          <creation_ts>2017-04-04 21:33:25 +0000</creation_ts>
          <short_desc>create-pull-request: add in-reply-to option to ensure patch/series threads don&apos;t break</short_desc>
          <delta_ts>2017-05-24 16:44:06 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>OE-Core</product>
          <component>Scripts and Tools</component>
          <version>2.4</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.4 M1</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jose Lamego">jose.a.lamego</reporter>
          <assigned_to name="Jose Lamego">jose.a.lamego</assigned_to>
          <cc>bluelightning</cc>
    
    <cc>leonardo.sandoval.gonzalez</cc>
    
    <cc>ross.burton</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Don&apos;t know</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>72088</commentid>
    <comment_count>0</comment_count>
    <who name="Jose Lamego">jose.a.lamego</who>
    <bug_when>2017-04-04 21:33:25 +0000</bug_when>
    <thetext>The create-patch-request script creates patches as replies to a cover letter, in the form of an email thread. If further revisions are sent to the mailing list without referencing to the first revision, these new revisions are not identified at the mailing list as part of the original thread, but as a new thread instead.

This behavior may not be seen as an issue at the mailing list, but at the OE-Patchwork tool, where any patch or series identified as part of a thread is added to the existing series in the form of a new revision, missing references may cause to have broken threads, thus producing individual series for revisions of the same series (this is specially true if the new revision differs in patch quantity or title from the original).

The proposed solution is to add the &quot;[-t in_reply_to]&quot; option to the create-pull-request script (where &quot;in_reply_to&quot; is the original cover letter&apos;s Message-Id), to ensure that the reference to the original thread is added to the new patch/series so it can be identified as part of it. With this option, the cover letter for any new revision is created as reply to the original cover letter, thus ensuring that OE-Patchwork will recognize the new series as a revision of an existing one.

The &quot;-t&quot; option would not be required, so the current script functionality would remain the same if &quot;-t&quot; is missing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>72090</commentid>
    <comment_count>1</comment_count>
    <who name="Jose Lamego">jose.a.lamego</who>
    <bug_when>2017-04-04 21:46:28 +0000</bug_when>
    <thetext>Proposed patch can be viewed at:
http://lists.openembedded.org/pipermail/openembedded-core/2017-April/135165.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>72106</commentid>
    <comment_count>2</comment_count>
    <who name="Jose Lamego">jose.a.lamego</who>
    <bug_when>2017-04-05 19:44:45 +0000</bug_when>
    <thetext>Patch v2 can be seen at:
http://lists.openembedded.org/pipermail/openembedded-core/2017-April/135237.html

thanks for the feedback Leo.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>73471</commentid>
    <comment_count>3</comment_count>
    <who name="Jose Lamego">jose.a.lamego</who>
    <bug_when>2017-05-24 16:44:06 +0000</bug_when>
    <thetext>The requested functionality is provided in commit:
f7c2ec9f8719ee37bdca866c881ca5d853f9aefc</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>