On Wed, Oct 1, 2008 at 15:21, Nikolai Weibull <now@bitwi.se> wrote:

> On Wed, Oct 1, 2008 at 11:38, Oliver Kiddle <okiddle@yahoo.co.uk> wrote:

>> Nikolai Weibull wrote:

>>> 1.  It seems that my trick to disable --force and --recursive on
>>> non-GNU systems doesn't work.

>> This is better done as:
>>  args=( ${args:#*--(force|recursive)\[*} )

> Thanks, will change to that.

Done.

>> The right way to do this would be to work out exactly what can't be
>> completed and build up a suitable glob pattern to pass to _files with
>> -g.
>
> Ugh.  That sounds complicated.

Done.

>> One other point: I really hope that running rm --version can't possibly
>> actually result in a file or files being removed on some system. I'm a
>> bit uneasy about it. Perhaps you should try using rm --help instead
>> because there's no `r' in help.

> OK, sounds reasonable.

Done.

How about including it now?