PATCH: Add dry run option to ipurge

Bron Gondwana brong at fastmail.fm
Tue Jun 7 19:39:21 EDT 2016


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.

Bron.

On Tue, Jun 7, 2016, at 23:24, Valentin Vidic via Cyrus-devel wrote:
> Hi,
> 
> It would be useful for this destructive command to have a dry run
> mode displaying what would be deleted.  The patch attached adds
> a -n option for this purpose.  The info displayed is similar to
> unexpunge output but without the cached info (not sure if it was
> useful and safe to call cache here).  Let me know what you think.
> 
> https://lists.andrew.cmu.edu/pipermail/info-cyrus/2014-August/037528.html
> 
> -- 
> Valentin
> Email had 1 attachment:
> + ipurge-dryrun.patch
>   6k (text/x-diff)


-- 
  Bron Gondwana
  brong at fastmail.fm


More information about the Cyrus-devel mailing list