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