GNU bug report logs - #6547
GNU coreutils 8.5 "make check" => tests/test-suite.log 8 of 365 tests failed

Previous Next

Full log

Message #59 received at 6547 <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: Pádraig Brady <P <at> draigBrady.com>
Cc: 6547 <at> debbugs.gnu.org
Subject: Re: bug#6547: GNU coreutils 8.5 "make check" => tests/test-suite.log
	8	of 365 tests failed
Date: Thu, 02 Sep 2010 14:50:12 +0200

Pádraig Brady wrote:
> On 02/09/10 12:39, Jim Meyering wrote:
>> Regarding your proposed new test, there is a slightly
>> more efficient way to do it:
>>
>>   $ git grep df.--local tests
>>   tests/du/basic:if df --local . >/dev/null 2>&1; then
>>   tests/du/slink:if df --local . >/dev/null 2>&1; then
>>   ...
>>
>> I.e., you can examine merely the exit status.
>> If you do that, please encapsulate the test in a function
>> to be defined in tests/init.cfg, I suppose.
>
> OK cool,
> I'll apply something like the following untested patch later.
...
>     tests: exclude some tests when running on NFS
>
>     * tests/init.cfg (is_local_dir_): A new function
>     returning if the current directory is on a local file system.
>     (require_local_dir_): A new function to skip tests
>     if not on a local dir.
>     * tests/cp/existing-perm-race: Skip if non local.
>     * tests/cp/file-perm-race: Likewise.
>     * tests/cp/parent-perm: Likewise.
>     * tests/cp/parent-perm-race: Likewise.
>     * tests/cp/preserve-2: Likewise.
>     * tests/mv/part-symlink: Likewise.
>     * tests/du/basic: Use refactored function.
>     * tests/du/slink: Likewise.

Thanks!  That looks right.
However at least 3 more tests can use one of the new functions
in place of existing df --local uses:
(yeah, depending on your font size, "..." without the quotes,
can be almost invisible)

  $ git grep -l df.--local tests
  tests/du/basic
  tests/du/slink
  tests/install/basic-1
  tests/ls/readdir-mountpoint-inode
  tests/mkdir/p-3

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.