autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: bug#10819: [BUG][RM]


From: Eric Blake
Subject: Re: bug#10819: [BUG][RM]
Date: Sat, 18 Feb 2012 08:49:34 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0) Gecko/20120131 Thunderbird/10.0

On 02/16/2012 02:19 PM, Stefano Lattarini wrote:

>> Does this look reasonable, then?
>>
>>      docs: tweak 'rm -f' limitations
>>      * doc/autoconf.texi (Limitations of Usual Tools) <rm>: More
>>      details on 'rm -f' without files.
>>

>> +It is not portable to invoke @command{rm} without options or operands.
>> +These days,
>>
> Adding an "OTOH" here would make the text clearer for me:
> 
>  "On the other hand, these days ..."
> 
> At which point ...
> 
>> +Posix requires @command{rm -f} (but not @command{rm} in
>> +isolation)
>>
> ... the aside in parentheses become redundant.
> 

> The rest looks good to me.

Thanks for the review.  I made those adjustments, then pushed.

-- 
Eric Blake   address@hidden    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]