From 21bdae12e11ae20460715475d8a0c991f15464ac Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Sat, 2 Jul 2011 01:44:49 +0200 Subject: [PATCH] tmpfiles: don't exit with an error code if we cannot access all files This is just to avoid confusion if people use stuff like FUSE file systems or SELinux which might disallow access to files even if tmpfiles is running as root. We still log away if we cannot access a file, but we do not return a failure exit code in the end. --- TODO | 4 ++++ src/tmpfiles.c | 8 ++------ 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/TODO b/TODO index 92c90eec..b999ce16 100644 --- a/TODO +++ b/TODO @@ -71,6 +71,8 @@ Features: * support notifications for services being enabled/disabled +* add support for /bin/mount -s + * GC unreferenced jobs (such as .device jobs) * add JoinControllers= to system.conf to mount certain cgroup @@ -180,6 +182,8 @@ Features: * detect LXC environment +* investigate whether the gnome pty helper should be moved into systemd, to provide cgroup support. + * Maybe store in unit files whether a service should be enabled by default on package installation (belongs into a distro pattern though, not in an upstream package's service file) diff --git a/src/tmpfiles.c b/src/tmpfiles.c index 954c3b7c..4394f0dd 100644 --- a/src/tmpfiles.c +++ b/src/tmpfiles.c @@ -983,15 +983,11 @@ int main(int argc, char *argv[]) { strv_free(files); } - - HASHMAP_FOREACH(i, globs, iterator) - if (process_item(i) < 0) - r = EXIT_FAILURE; + process_item(i); HASHMAP_FOREACH(i, items, iterator) - if (process_item(i) < 0) - r = EXIT_FAILURE; + process_item(i); finish: while ((i = hashmap_steal_first(items))) -- 2.39.5