From: Shawn Pearce <spearce@spearce.org>
Date: Tue, 21 Feb 2006 22:32:54 GMT
Subject: Re: [PATCH] Add new git-rm command with documentation
Message-ID: <20060221223254.GB20744@spearce.org>
In-Reply-To: <43FB9455.6010402@people.pl>

Krzysiek Pawlik <krzysiek.pawlik@people.pl> wrote:
[...]
> +while : ; do
> +  case "$1" in
> +    -n)
> +	show_only=true
> +	;;
> +    -v)
> +	verbose=--verbose
> +	;;
> +    -f)
> +	remove_files=true
> +	;;
> +    --)
> +	break
> +	;;
> +    -*)
> +	usage
> +	;;
> +    *)
> +	break
> +	;;
> +  esac
> +  shift
> +done
[...]

You are leaving -- in $@ for processing later, which means we'll
try to delete the file '--'.  :-)

I think a shift before the break in the -- case would fix this.

-- 
Shawn.