Re: [PATCH] Make translation status report message threaded

On Thu, Mar 10, 2011 at 17:54, Philip Martin <philip.martin_at_wandisco.com> wrote:
> Dongsheng Song <dongsheng_at_apache.org> writes:
>
>> Index: l10n-report.py
>> ===================================================================
>> --- l10n-report.py      (revision 1080141)
>> +++ l10n-report.py      (working copy)
>> @@ -43,6 +43,7 @@
>>  FROM_ADDRESS = "Subversion Translation Status <noreply_at_subversion.apache.org>"
>>  LIST_ADDRESS = "dev_at_subversion.apache.org"
>>  SUBJECT_TEMPLATE = "[l10n] Translation status report for %s r%s"
>> +MAIL_THREAD_ID = '<translation_status_report_for_%s_at_subversion.apache.org>'
>>
>>  def _rev():
>>    dollar = "$Revision$"
>> @@ -204,6 +205,8 @@
>>          msg["X-Mailer"] = "l10n-report.py r%s" % _rev()
>>          msg["Reply-To"] = LIST_ADDRESS
>>          msg["Mail-Followup-To"] = LIST_ADDRESS
>> +        msg["In-Reply-To"] = MAIL_THREAD_ID % (branch_name.replace('/', '_'))
>> +        msg["References"] = msg["In-Reply-To"]
>>          # http://www.iana.org/assignments/auto-submitted-keywords/auto-submitted-keywords.xhtml
>>          msg["Auto-Submitted"] = 'auto-generated'
>>          msg.set_type("text/plain")
>
> In-Reply-To and References would usually contain message-ids, what you
> are inserting is a fake message-id -- no email has it.  A MUA won't find
> the referenced message-id, it doesn't exist, but will see multiple
> messages that refer to that id.  I guess lots of MUAs will react by
> grouping those messages together.  Is that what you want to achieve?
>
> --
> Philip
>

Do you mean the following behavior ?

http://people.apache.org/~dongsheng/image/public/mail_report_threaded.png

This is exactly what I needed.

--
Dongsheng