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   ebl...@redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to