PATCH: Add dry run option to ipurge
ellie timoney
ellie at fastmail.com
Thu Jun 9 22:19:35 EDT 2016
Hi again,
This is now on the master and cyrus-imapd-2.5 branches, thanks again!
Thanks too for including relevant documentation updates in your patches,
it's very much appreciated.
Cheers,
ellie
On Fri, Jun 10, 2016, at 11:09 AM, ellie timoney via Cyrus-devel wrote:
> Hi Valentin,
>
> We spoke briefly on IRC a few days ago. Thanks for sending the patches
> through.
>
> I'm looking at them now -- all going well, they'll be merged sometime
> today. :)
>
> Cheers,
>
> ellie
>
> On Wed, Jun 8, 2016, at 04:44 PM, Valentin Vidic via Cyrus-devel wrote:
> > On Wed, Jun 08, 2016 at 09:39:21AM +1000, Bron Gondwana via Cyrus-devel
> > wrote:
> > > Looks good to me. It's perfectly safe to load the cache data as well if we
> > > want to extend it to have more information later, but it's definitely worth
> > > having. I'd say it's a candidate for 2.5 and master.
> >
> > Great, I'm sending an updated patch using the cache info. It also fixes
> > the
> > stats capitalization for total.
> >
> > --
> > Valentin
> > Email had 1 attachment:
> > + ipurge-dryrun2.patch
> > 7k (text/x-diff)
More information about the Cyrus-devel
mailing list