summary refs log tree commit diff
path: root/pkgs/os-specific/linux/systemd/0017-core-don-t-taint-on-unmerged-usr.patch
blob: 73b237a29602bf7ade2f7564dcf037453539a9ac (plain) (blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: oxalica <oxalicc@pm.me>
Date: Tue, 4 Oct 2022 09:18:07 +0800
Subject: [PATCH] core: don't taint on unmerged /usr

NixOS has very different approach towards /bin and /sbin - they don't
really exist (except for /bin/sh and /usr/bin/env, because these are used
heavily in shebangs around the world). The concept of merged or unmerged
usr doesn't really apply here at all, it's neither of the two.
Users don't execute things from /bin or /sbin, there's nothing else in
there. In all cases, systemd doesn't look things up from /usr/bin or /bin,
so showing the taint isn't really helpful.

See also: https://github.com/systemd/systemd/issues/24191
---
 src/core/manager.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/core/manager.c b/src/core/manager.c
index acf3ead8d7..bdbab16829 100644
--- a/src/core/manager.c
+++ b/src/core/manager.c
@@ -4754,10 +4754,6 @@ char* manager_taint_string(const Manager *m) {
         if (m->taint_usr)
                 stage[n++] = "split-usr";
 
-        _cleanup_free_ char *usrbin = NULL;
-        if (readlink_malloc("/bin", &usrbin) < 0 || !PATH_IN_SET(usrbin, "usr/bin", "/usr/bin"))
-                stage[n++] = "unmerged-usr";
-
         if (access("/proc/cgroups", F_OK) < 0)
                 stage[n++] = "cgroups-missing";