Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: >> > +die_abort () { >> > + rm -rf "$DOTEST" 2> /dev/null >> > + die "$1" >> > +} >> >> Why "2>/dev/null" here? > > Just to be sure. If it does not exist, it's no error. No sense alarming > the user. Yeah, but isn't that exactly why you have "-f" there? On the other hand, if $DOTEST exists but for some reason couldn't be removed, we probably would want to know about it. $ rm -fr no-such $ mkdir no-such $ echo >no-such/file $ rm -fr no-such $ mkdir no-such $ echo >no-such/file $ chmod a-w no-such $ rm -fr no-such rm: cannot remove `no-such/file': Permission denied - 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