Eric Lesh <eclesh@xxxxxxxx> writes:
> Signed-off-by: Eric Lesh <eclesh@xxxxxxxx>
>
> ---
>
> On Sun, 2007-03-25 at 23:36 +0200, Johannes Schindelin wrote:
>> > Too bad, I find it rather annoying and irritating.
>>
>> Why not do the common thing, and add a "--quiet" option? You can even add
>> a config variable to enable it by default (for git-rm). It's not like
>> git-rm is performance critical...
>
> Is something like this right?
>
> builtin-rm.c | 7 +++++--
> 1 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/builtin-rm.c b/builtin-rm.c
> index 00dbe39..d193fb0 100644
> --- a/builtin-rm.c
> +++ b/builtin-rm.c
> @@ -114,7 +114,7 @@ static struct lock_file lock_file;
> int cmd_rm(int argc, const char **argv, const char *prefix)
> {
> int i, newfd;
> - int show_only = 0, force = 0, index_only = 0, recursive = 0;
> + int show_only = 0, force = 0, index_only = 0, recursive = 0, quiet = 0;
> const char **pathspec;
> char *seen;
>
> @@ -197,7 +199,8 @@ int cmd_rm(int argc, const char **argv, const char *prefix)
> */
> for (i = 0; i < list.nr; i++) {
> const char *path = list.name[i];
> - printf("rm '%s'\n", path);
> + if (!quiet)
> + printf("rm '%s'\n", path);
>
> if (remove_file_from_cache(path))
> die("git-rm: unable to remove %s", path);
I wonder how this would interact with show_only...
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html