Skip to content

Commit e5fb601

Browse files
committed
Move meminfo-writer to /usr/bin
There it a tendency to merge /usr/bin and /usr/sbin (replace the latter with a symlink). This caused a few issues already, including SELinux mislabeling (QubesOS/qubes-issues#9663), file install issues where %_sbindir RPM macro points at /usr/bin, and now on in-place upgrade systemd unit points at /usr/sbin/meminfo-writer that doesn't exist at the upgrade time. Since this merging happened on Fedora 42 now, and much earlier on Archlinux too, simplify things by simply moving meminfo-writer to /usr/bin and don't have any special cases for distributions with merged /usr/sbin or not. The change isn't relevant for Debian (yet?) but also shouldn't hurt either. QubesOS/qubes-issues#9807
1 parent 6578e70 commit e5fb601

6 files changed

Lines changed: 6 additions & 7 deletions

File tree

debian/qubes-utils.install

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
usr/sbin/meminfo-writer
1+
usr/bin/meminfo-writer
22
lib/systemd/system/qubes-meminfo-writer.service
33
usr/lib/qubes/*
44
usr/lib/udev/*

qmemman/Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
CC=gcc
22
CFLAGS+=-Wall -Wextra -Werror -g -O3
33
all: meminfo-writer
4-
SBINDIR?=/usr/sbin
4+
BINDIR?=/usr/bin
55

66
_XENSTORE_H=$(shell ls /usr/include/xenstore.h)
77
ifneq "$(_XENSTORE_H)" ""
@@ -11,7 +11,7 @@ endif
1111
meminfo-writer: meminfo-writer.o
1212
$(CC) $(LDFLAGS) -g -o meminfo-writer meminfo-writer.o -lxenstore
1313
install:
14-
install -D meminfo-writer $(DESTDIR)/$(SBINDIR)/meminfo-writer
14+
install -D meminfo-writer $(DESTDIR)/$(BINDIR)/meminfo-writer
1515
ifeq (1,${DEBIANBUILD})
1616
install -d $(DESTDIR)/lib/systemd/system/
1717
install -m 0644 qubes-meminfo-writer.service $(DESTDIR)/lib/systemd/system/

qmemman/qubes-meminfo-writer-dom0.service

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ ConditionPathExists=/run/qubes/qmemman.sock
66

77
[Service]
88
Type=simple
9-
ExecStart=/usr/sbin/meminfo-writer 30000 100000
9+
ExecStart=/usr/bin/meminfo-writer 30000 100000
1010
RemainAfterExit=yes
1111

1212
[Install]

qmemman/qubes-meminfo-writer.service

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ Before=systemd-user-sessions.service
55

66
[Service]
77
Type=forking
8-
ExecStart=/usr/sbin/meminfo-writer 30000 100000 /run/meminfo-writer.pid
8+
ExecStart=/usr/bin/meminfo-writer 30000 100000 /run/meminfo-writer.pid
99
PIDFile=/run/meminfo-writer.pid
1010

1111
[Install]

rpm_spec/qubes-utils.spec.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ rm -rf $RPM_BUILD_ROOT
132132
%_tmpfilesdir/xen-devices-qubes.conf
133133
%dir %{_prefix}/lib/qubes
134134
%{_prefix}/lib/qubes/udev-*
135-
%{_sbindir}/meminfo-writer
135+
%{_bindir}/meminfo-writer
136136
%{_unitdir}/qubes-meminfo-writer.service
137137
%{_unitdir}/qubes-meminfo-writer-dom0.service
138138
%dir %_includedir/qubes

selinux/qubes-meminfo-writer.fc

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,2 @@
11
/usr/bin/meminfo-writer -- gen_context(system_u:object_r:qubes_meminfo_writer_exec_t,s0)
2-
/usr/sbin/meminfo-writer -- gen_context(system_u:object_r:qubes_meminfo_writer_exec_t,s0)
32
/run/meminfo-writer\.pid -- gen_context(system_u:object_r:qubes_meminfo_writer_var_run_t,s0)

0 commit comments

Comments
 (0)