[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: new snapshot available: coreutils-8.17.70-1bacb4
From: |
Jim Meyering |
Subject: |
Re: new snapshot available: coreutils-8.17.70-1bacb4 |
Date: |
Wed, 08 Aug 2012 18:33:02 +0200 |
Pádraig Brady wrote:
> 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 :)
Yep. I should have looked at the code.
> Updated version that explicitly matches the error is attached.
...
> Subject: [PATCH] tests: fix validation of local file systems
>
> * tests/init.cfg (require_mount_list_): A new function
> to ensure we can read the list of file systems.
> (require_local_dir_): Call the above function as otherwise
s/ as/, as/ :-)
> the check is invalid.
> * tests/df/total-unprocessed: Ensure df can read the
> list of mounted file systems so that --local can be honored.
...
> +require_mount_list_()
> +{
> + mount_list_fail='cannot read table of mounted file systems'
Might as well get in the habit of using "local" in this file.
> + df 2>&1 | grep -F "$mount_list_fail" >/dev/null &&
> + skip_ "$mount_list_fail"
> +}
> +
> require_local_dir_()
> {
> + require_mount_list_
> is_local_dir_ . ||
> skip_ "This test must be run on a local file system."
> }
ACK.
- new snapshot available: coreutils-8.17.70-1bacb4, Jim Meyering, 2012/08/08
- Re: new snapshot available: coreutils-8.17.70-1bacb4, Pádraig Brady, 2012/08/08
- Re: new snapshot available: coreutils-8.17.70-1bacb4, Jim Meyering, 2012/08/08
- Re: new snapshot available: coreutils-8.17.70-1bacb4, Bernhard Voelker, 2012/08/08
- Re: new snapshot available: coreutils-8.17.70-1bacb4, Jim Meyering, 2012/08/08
- Re: new snapshot available: coreutils-8.17.70-1bacb4, Bernhard Voelker, 2012/08/08
- Re: new snapshot available: coreutils-8.17.70-1bacb4, Pádraig Brady, 2012/08/08
- Re: new snapshot available: coreutils-8.17.70-1bacb4, Bernhard Voelker, 2012/08/08
- Re: new snapshot available: coreutils-8.17.70-1bacb4,
Jim Meyering <=
- [PATCH] df: fail when the mount list cannot be read but is needed [was: new snapshot available: coreutils-8.17.70-1bacb4], Bernhard Voelker, 2012/08/09
- Re: [PATCH] df: fail when the mount list cannot be read but is needed [was: new snapshot available: coreutils-8.17.70-1bacb4], Bernhard Voelker, 2012/08/14
- Re: [PATCH] df: fail when the mount list cannot be read but is needed [was: new snapshot available: coreutils-8.17.70-1bacb4], Jim Meyering, 2012/08/14
- Re: [PATCH] df: fail when the mount list cannot be read but is needed [was: new snapshot available: coreutils-8.17.70-1bacb4], Bernhard Voelker, 2012/08/14
- Re: [PATCH] df: fail when the mount list cannot be read but is needed [was: new snapshot available: coreutils-8.17.70-1bacb4], Jim Meyering, 2012/08/14
- Re: [PATCH] df: fail when the mount list cannot be read but is needed [was: new snapshot available: coreutils-8.17.70-1bacb4], Bernhard Voelker, 2012/08/14