Re: [PATCH] Have --format=mbox insert Status:

[Peter John Hartman <peterjohnhartman@gmail.com>]

On Mon, Feb 07, 2011 at 10:04:41AM +0100, Michal Sojka wrote:
> > [mbox]
> > synchronize_flags=true
> 
> I would call this differently - perhaps 'export_flags'. In maildir
> section of config file synchronize_flags has the meaning of
> bi-directional synchronization and this is only uni-directional.

I agree.

> 
> > xstatus_A=replied
> > xstatus_F=flagged
> > xstatus_T=draft
> > #xstatus_D=deleted
> > status_R=-unread
> > status_O=unread
> 
> Is it necessary for this to be configurable? Given the list bellow, it
> seems it is already defined by the "standard", so users would rarely
> want different mapping.

I agree.  In fact, I would think we don't need to add anything to the
notmuch config file at all.  I can't think of a use case where we'd want
show --format=mbox to *not* also replicate/export the Status/X-Status tags.
I personally don't use X-Status: but I don't have a problem with my patch
being patched to include them.  It won't cause any problems.

Peter


-- 
sic dicit magister P
PhD Candidate
Collaborative Programme in Ancient and Medieval Philosophy
University of Toronto
http://individual.utoronto.ca/peterjh