Re: [perf/core PATCH v2 4/4] perf buildid-cache: Add --remove-all FILE to remove all caches of FILE

2015-02-12 Thread Arnaldo Carvalho de Melo
Em Thu, Feb 12, 2015 at 03:13:02PM +0900, Masami Hiramatsu escreveu: > (2015/02/12 0:00), Arnaldo Carvalho de Melo wrote: > > Em Tue, Feb 10, 2015 at 06:18:58PM +0900, Masami Hiramatsu escreveu: > >> Add --remove-all FILE to remove all caches of FILE. > > > > I like --purge better, shorter form, l

Re: [perf/core PATCH v2 4/4] perf buildid-cache: Add --remove-all FILE to remove all caches of FILE

2015-02-11 Thread Masami Hiramatsu
(2015/02/12 0:00), Arnaldo Carvalho de Melo wrote: > Em Tue, Feb 10, 2015 at 06:18:58PM +0900, Masami Hiramatsu escreveu: >> Add --remove-all FILE to remove all caches of FILE. > > I like --purge better, shorter form, like in 'man fpurge' Thanks for good suggestion :) I'll update this. Thank yo

Re: [perf/core PATCH v2 4/4] perf buildid-cache: Add --remove-all FILE to remove all caches of FILE

2015-02-11 Thread Arnaldo Carvalho de Melo
Em Tue, Feb 10, 2015 at 06:18:58PM +0900, Masami Hiramatsu escreveu: > Add --remove-all FILE to remove all caches of FILE. I like --purge better, shorter form, like in 'man fpurge' - Arnaldo > Since the current --remove FILE removes a cache which has > same build-id of given FILE. Since the comm

[perf/core PATCH v2 4/4] perf buildid-cache: Add --remove-all FILE to remove all caches of FILE

2015-02-10 Thread Masami Hiramatsu
Add --remove-all FILE to remove all caches of FILE. Since the current --remove FILE removes a cache which has same build-id of given FILE. Since the command takes a FILE path, it can confuse user who tries to remove cache about FILE path. - # ./perf buildid-cache -v --add ./perf Adding 1