Bug 62859

Summary: test-unit-file: fails due to assertion
Product: systemd Reporter: Michał Górny <mgorny>
Component: generalAssignee: systemd-bugs
Status: RESOLVED WONTFIX QA Contact: systemd-bugs
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:

Description Michał Górny 2013-03-28 10:43:36 UTC
./build-aux/test-driver: line 95: 31794 Aborted                 "$@" > $log_file 2>&1
FAIL: test-unit-file

$ cat test-unit-file.log 
unit_file_get_list: Permission denied
Assertion 'r >= 0' failed at src/test/test-unit-file.c:50, function test_unit_file_get_set(). Aborting.

AFAICS it should be skipped instead due to insufficient permissions.
Comment 1 Zbigniew Jedrzejewski-Szmek 2013-08-04 04:05:00 UTC
Since http://cgit.freedesktop.org/systemd/systemd/commit/?id=751e757 we return error without dumping core. This is really a misconfiguration in the environment, so a legitimate failure.
Comment 2 Michał Górny 2013-08-04 07:48:31 UTC
What's exactly the 'misconfiguration'? Are people expected to run tests as root?
Comment 3 Zbigniew Jedrzejewski-Szmek 2013-08-04 12:36:12 UTC
(In reply to comment #2)
> What's exactly the 'misconfiguration'? Are people expected to run tests as
> root?
No, people are not expected to have unreadable unit files. systemd will serve their contents over dbus anyway, so it's unlikely that chmod'ding files does what people who do it expect.
Comment 4 Michał Górny 2013-08-04 13:00:37 UTC
Well, you could say straight that it fails because file in /etc/systemd is not world-readable. Not that tests which rely on system configuration of *build host* are any good...

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.