On Sun, 28 Mar 2010, Ben Gamari wrote: > On Sun, 28 Mar 2010 07:57:30 +0200, Michal Sojka <sojkam1@fel.cvut.cz> wrote: > > On Sun, 28 Mar 2010, Ben Gamari wrote: > > > On Fri, 12 Mar 2010 17:13:26 -0500, Ben Gamari <bgamari.foss@gmail.com> wrote: > > > > Add a new_tags option in the [messages] section of the configuration > > > > file to allow the user to specify which tags should be added to new > > > > messages by notmuch new. > > > > > > Would anyone oppose this being merged? I've been running with it on a daily > > > basis for the last two weeks and I've had no issues. The migration path is > > > quite clean as the current behavior is the default. > > > > Hi, > > > > I think that your patch is only a temporary solution. Some people > > requests the ability to tag messages based on the maildir flags so the > > question is how would your patch integrate with maildir-based tagging? > > > I would argue that maildir-tagging and basic notmuch new tagging could be > orthogonal. I think maildir flag tagging should be handled completely by the > maildir mail store backend. I could be wrong though. > > In fact, now that I think of it, perhaps this patch's functionality belongs in > the mail store backend as well. I haven't tried your patchset, but it looks > like it would be trivial to add this as a mailstore option. I agree. I'll try to add your patch to my mailstore patch set. It will be enabled only when the default mailstore (called files) is enabled. I'll move the config option from [messages] to [mailstore] section and rename it from new_tags to files.new_tags, to stress the fact, that it applies only for files backed. Do you agree? Michal