Summary: | sys-apps/coreutils fails test ls/stat-free-color with sandbox due to using `strace` and counting stat calls | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Toralf Förster <toralf> |
Component: | [OLD] Core system | Assignee: | Gentoo's Team for Core System packages <base-system> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | erikdenstore+gbugs |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | build log |
Description
Toralf Förster
2012-05-11 14:35:47 UTC
Created attachment 311433 [details]
build log
The test counts the difference in the number of stat calls between "ls --help" and "ls --color=always .", and checks the difference is at most 2. That is: it checks that even with color output enabled ls can do its thing without statting every file in the directory. This test fails when run manually with sandbox, and succeeds without it. So it's just failing because sandbox adds some extra stat calls. I doubt this can be fixed sensibly: the test should just be skipped (at least if sandbox is enabled). I know of no sane way to only count stat calls from ls itself, and not ones added by sandbox. And adjusting the expected output would be way too fragile: the number of stat calls sandbox needs is going to vary between sandbox versions. To run just this test, do this in ${S}/tests/: make check TESTS=ls/stat-free-color VERBOSE=yes You should see running the above through "sandbox" fails. *** Bug 494654 has been marked as a duplicate of this bug. *** i've updated the test patch to skip this test http://sources.gentoo.org/gentoo/src/patchsets/coreutils/8.22/010_all_coreutils-tests.patch?r1=1.1&r2=1.2 The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0edff67f453a7509c08fce538d3da9c947daa433 commit 0edff67f453a7509c08fce538d3da9c947daa433 Author: Thomas Deutschmann <whissi@gentoo.org> AuthorDate: 2017-11-06 16:20:43 +0000 Commit: Thomas Deutschmann <whissi@gentoo.org> CommitDate: 2017-11-06 16:20:57 +0000 sys-apps/coreutils: Rev bump to address multiple test failures - Running test suite with FEATURES=usersandbox is no longer supported due to various sandbox issues. (bug #413621, #439574, #629660) - Due to sandbox failures previously disabled tests re-enabled. (bug #415487) - tests/ls/readdir-mountpoint-inode test failure fixed. (bug #353164) - "caps" USE flag description more specified. (bug #617702 - Tools/programs/libraries used in test suite now listed in DEPEND="test? (...)" which requires re-keywording for some architectures. Bug: https://bugs.gentoo.org/413621 Bug: https://bugs.gentoo.org/439574 Bug: https://bugs.gentoo.org/629660 Bug: https://bugs.gentoo.org/415487 Closes: https://bugs.gentoo.org/353164 Closes: https://bugs.gentoo.org/617702 Package-Manager: Portage-2.3.13, Repoman-2.3.4 sys-apps/coreutils/Manifest | 1 + sys-apps/coreutils/coreutils-8.28-r1.ebuild | 210 ++++++++++++++++++++++++++++ sys-apps/coreutils/metadata.xml | 1 + 3 files changed, 212 insertions(+)} |