Messages in this thread | | | Date | Tue, 2 Dec 2014 13:25:18 +0100 | From | Jiri Olsa <> | Subject | Re: [PATCH 0/8] perf buildid-cache: Add clean command |
| |
On Mon, Dec 01, 2014 at 08:29:19PM -0200, Arnaldo Carvalho de Melo wrote:
SNIP
> [acme@sandy wb]$ > > If we keep the way it is, I think it would be clearer to add a -l/--list > command and that would: > > perf buildid-cache --list # list all the files in the cache > perf buildid-cache --list 3d # list 3 days old files > perf buildid-cache --list +3d # list files 3 days or more older > perf buildid-cache --list -3d # list files up to 3 days old
ok, I was up to old files (and big files) removal only.. not sure the other way will be actually needed.. but it's easy enought to add
> > And all these would print the sum of the matching files, i.e. the space > those files use in the filesystem. > > > Remove cache files older than 3 days: > > $ perf buildid-cache clean 3d -r > > perf buildid-cache --remove [-+]Nd # Same semantics as above > > > > > Total cache removal: > > $ perf buildid-cache clean -r > > perf buildid-cache --remove '*'
hum, maybe '--removal all' might be better
> > > Remove and display items bigger than 200M > > $ perf buildid-cache clean -r -a 200M > > But yeah, perhaps we could switch to having subcommands and add: > > perf buildid-cache [add|remove|update|list] > > What do you think?
yep, those commands seem nicer..
let's see of there're other comments and I'll come up with new interface after..
thanks, jirka
| |