From: Steven Grimm <koreth@midwinter.com>
To: Junio C Hamano <junkio@cox.net>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] Add -q option to "git rm" to suppress output when there aren't errors.
Date: Sun, 15 Apr 2007 17:17:48 -0700 [thread overview]
Message-ID: <4622C0AC.8090904@midwinter.com> (raw)
In-Reply-To: <7v4pnh18hr.fsf@assigned-by-dhcp.cox.net>
Junio C Hamano wrote:
> Suppressing output is understandable and probably is a useful
> thing to do, but I do not see a justification to tie that
> quietness to making the status unuable...
>
The status is unusable as is, actually, for the particular use case of
cg-admin-rewritehist. If you try to use git-rm as an index filter,
cg-admin-rewritehist will stop running as soon as you hit a revision
that doesn't contain the file you're looking to filter out. (If the file
doesn't exist in the first revision in your repo, that means it will do
no work at all.)
My justification was that the "-f" option to the normal UNIX "rm"
command does the same thing: it suppresses output and causes the command
to exit with a zero status code even if no files match. But "git rm -f"
was already taken.
If you like I'll be happy to split it into two separate options.
-Steve
next prev parent reply other threads:[~2007-04-16 0:17 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-04-16 0:04 [PATCH] Add -q option to "git rm" to suppress output when there aren't errors Steven Grimm
2007-04-16 0:13 ` Junio C Hamano
2007-04-16 0:17 ` Steven Grimm [this message]
2007-04-16 1:14 ` Junio C Hamano
2007-04-16 7:46 ` [PATCH 1/2] Add --quiet option to suppress output of "rm" commands for removed files Steven Grimm
2007-04-16 7:53 ` [PATCH 2/2] Add --ignore-notfound option to exit with zero status when no files are removed Steven Grimm
2007-04-16 7:59 ` Junio C Hamano
2007-04-16 8:13 ` Steven Grimm
2007-04-16 8:50 ` Jeff King
2007-04-16 8:53 ` Junio C Hamano
2007-04-16 9:04 ` Jeff King
2007-04-16 18:29 ` Josef Weidendorfer
2007-04-16 8:12 ` Steven Grimm
2007-04-16 7:54 ` [PATCH] Add -q option to "git rm" to suppress output when there aren't errors Alex Riesen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4622C0AC.8090904@midwinter.com \
--to=koreth@midwinter.com \
--cc=git@vger.kernel.org \
--cc=junkio@cox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).