Hello, Thanks for the comments, these have been addressed in the attached patch.
I discussed about this with Santiago and we think it makes sense for the patch to be added as part of this group of changesets. Please let us know how can we help. I'm looking forward for these new features, thanks! Yoshiki. On (Jan-28-15|10:25), Kevin J. McCarthy wrote: > David Champion wrote: > > * On 27 Jan 2015, Yoshiki Vazquez Baeza wrote: > > > Hello, > > > > > > This patch adds a new setting that you can configure to execute a > > > command every time new mail is found in your current mailbox. > > > > I've had a couple of patches in progress for a while that are related. > > I wanted to mention it mainly because I've already done most of that > > work, and it should not be seen as a comment on your code -- just two > > different but related approaches to the same problem, which can coexist. > > I wanted to propose it for discussion, but now that your patch is here > > the discussion has begun without the code. :) So this message is for > > Yoshiki to respond to, but also for general community response to a > > design idea for mail notification. > > Hi David, > > I just wanted to add my voice of support for the idea. I think these > two patches could integrate well with each other, and would personally > vote for committing them at the same time, with some kind of shared > notification syntax. > > I don't want to speak for Yoshiki or Santiago, but I would guess it > would be helpful to give some pointers as to where to integrate, or > perhaps do the integration but give them credit in the patch. > > -Kevin
