Steven Grimm <koreth@xxxxxxxxxxxxx> writes: > Documentation/git-rm.txt | 3 +++ > builtin-rm.c | 21 +++++++++++++++++---- > t/t3600-rm.sh | 4 ++++ > 3 files changed, 24 insertions(+), 4 deletions(-) I find it easier to see what is going on with two independent features separated out, especially with tests. Much nicer. > @@ -47,6 +47,9 @@ OPTIONS > the paths only from the index, leaving working tree > files. > > +\--ignore-notfound:: > + Exit with a zero status even if no files matched. > + ls-files has --error-unmatch so we may want to make the name consistent by saying --ignore-unmatch? I particularly do not care about the minute naming issues _right_ _now_, but we might regret it in the long run. No need to resend unless you have an actual fix; I'll queue your patch as is in 'pu', wait for a few days for comments on the naming, and amend it as needed. Thanks. - 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