Re: new snapshot available: coreutils-8.17.70-1bacb4

[Pádraig Brady]

On 08/08/2012 05:07 PM, Jim Meyering wrote:
> Bernhard Voelker wrote:
> 
>> On 08/08/2012 05:43 PM, Pádraig Brady wrote:
>>
>>> +require_mount_list_()
>>> +{
>>> +  df >/dev/null 2>&1 ||
>>> +    skip_ "Unable to read the list of mounted file systems."
>>> +}
>>> +
>>
>> Hmm, I think this is not enough: df can also fail due to other reasons,
>> e.g. when a mount point is not reachable by the user:
>>
>>   $ src/df > /dev/null  ; echo $?
>>   src/df: ‘/root/backup’: Permission denied
>>   1
> 
> Good point.
> 
> "df ." or "df /" should be more reliable.

But then it won't fail. It's tricky :)

Updated version that explicitly matches the error is attached.

cheers,
Pádraig.

df-tests.diff
Description: Text document