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 br...@fastmail.fm
