Pino Toscano
2017-Mar-10 14:09 UTC
[Libguestfs] [PATCH] appliance: run systemd-tmpfiles also for /var/run
Commit a6330e9d3af0f5286f1d53d909fd868387b67f69 enabled /run for
systemd-tmpfiles: while this works fine in most of the cases, there are
few tmpfiles configurations that still references /var/run instead of
/run. As result, include also /var/run in the systemd-tmpfiles
execution.
---
appliance/init | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/appliance/init b/appliance/init
index b951857..968429c 100755
--- a/appliance/init
+++ b/appliance/init
@@ -90,7 +90,7 @@ machine_id=$(dd if=/dev/urandom bs=16 count=1 status=none | od
-x -A n)
echo "${machine_id// /}" > /etc/machine-id
# Set up tmpfiles (must run after kmod.conf is created above).
-systemd-tmpfiles --prefix=/dev --prefix=/run --create --boot
+systemd-tmpfiles --prefix=/dev --prefix=/run --prefix=/var/run --create --boot
# Find udevd and run it directly.
for f in /lib/systemd/systemd-udevd /usr/lib/systemd/systemd-udevd \
--
2.9.3
Richard W.M. Jones
2017-Mar-10 21:44 UTC
Re: [Libguestfs] [PATCH] appliance: run systemd-tmpfiles also for /var/run
On Fri, Mar 10, 2017 at 03:09:46PM +0100, Pino Toscano wrote:> Commit a6330e9d3af0f5286f1d53d909fd868387b67f69 enabled /run for > systemd-tmpfiles: while this works fine in most of the cases, there are > few tmpfiles configurations that still references /var/run instead of > /run. As result, include also /var/run in the systemd-tmpfiles > execution. > --- > appliance/init | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/appliance/init b/appliance/init > index b951857..968429c 100755 > --- a/appliance/init > +++ b/appliance/init > @@ -90,7 +90,7 @@ machine_id=$(dd if=/dev/urandom bs=16 count=1 status=none | od -x -A n) > echo "${machine_id// /}" > /etc/machine-id > > # Set up tmpfiles (must run after kmod.conf is created above). > -systemd-tmpfiles --prefix=/dev --prefix=/run --create --boot > +systemd-tmpfiles --prefix=/dev --prefix=/run --prefix=/var/run --create --boot > > # Find udevd and run it directly. > for f in /lib/systemd/systemd-udevd /usr/lib/systemd/systemd-udevd \ > -- > 2.9.3Looks straightforward, ACK. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com Fedora Windows cross-compiler. Compile Windows programs, test, and build Windows installers. Over 100 libraries supported. http://fedoraproject.org/wiki/MinGW
Possibly Parallel Threads
- [PATCH 5/5] appliance: run systemd-tmpfiles also for /run
- [PATCH 4/4] appliance: run systemd-tmpfiles also for /run
- [PATCH v2 0/4] Improve LVM handling in the appliance
- [PATCH 0/5] Improve LVM handling in the appliance
- [PATCH 0/2] appliance: When running systemd-tmpfiles, pass --boot