public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "William Hubbs" <williamh@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] proj/openrc:openrc-0.11.x commit in: runlevels/, init.d/, conf.d/
Date: Wed, 31 Oct 2012 22:39:16 +0000 (UTC)	[thread overview]
Message-ID: <1351723121.c73173ae5389e1a82aaebf2b4503ec7af32052fb.WilliamH@OpenRC> (raw)

commit:     c73173ae5389e1a82aaebf2b4503ec7af32052fb
Author:     William Hubbs <w.d.hubbs <AT> gmail <DOT> com>
AuthorDate: Wed Oct 31 15:53:37 2012 +0000
Commit:     William Hubbs <williamh <AT> gentoo <DOT> org>
CommitDate: Wed Oct 31 22:38:41 2012 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=proj/openrc.git;a=commit;h=c73173ae

Tmpfiles: create and delete entries once in the boot runlevel.

Initially, we were creating tmpfiles entries in the sysinit runlevel and
again in the boot runlevel. Systemd runs the --create and --remove
options in one service called systemd-tmpfiles-setup after the local
file systems are mounted. Now we have a service called tmpfiles.setup
which emulates this.

This also closes the bug mentioned below, since we were originally
writing to files that were on read-only file systems and that were not
available.

Reported-by: <devurandom <AT> gmx.net>
X-Gentoo-Bug: 439012
X-Gentoo-Bug-URL: https://bugs.gentoo.org/show_bug.cgi?id=439012

---
 conf.d/Makefile                                 |    2 +-
 conf.d/{tmpfilesd => tmpfiles}                  |    0
 init.d/.gitignore                               |    3 +--
 init.d/Makefile                                 |    3 +--
 init.d/{tmpfilesd.boot.in => tmpfiles.setup.in} |    6 +++---
 init.d/tmpfilesd.sysinit.in                     |   20 --------------------
 runlevels/Makefile                              |    4 ++--
 7 files changed, 8 insertions(+), 30 deletions(-)

diff --git a/conf.d/Makefile b/conf.d/Makefile
index 1f28967..24db589 100644
--- a/conf.d/Makefile
+++ b/conf.d/Makefile
@@ -1,5 +1,5 @@
 DIR=	${CONFDIR}
-CONF=	bootmisc fsck hostname localmount netmount urandom tmpfilesd \
+CONF=	bootmisc fsck hostname localmount netmount urandom tmpfiles \
 		${CONF-${OS}}
 
 ifeq (${MKNET},)

diff --git a/conf.d/tmpfilesd b/conf.d/tmpfiles
similarity index 100%
rename from conf.d/tmpfilesd
rename to conf.d/tmpfiles

diff --git a/init.d/.gitignore b/init.d/.gitignore
index 515348b..c42cd77 100644
--- a/init.d/.gitignore
+++ b/init.d/.gitignore
@@ -41,5 +41,4 @@ syslogd
 termencoding
 ttys
 wscons
-tmpfilesd.boot
-tmpfilesd.sysinit
+tmpfiles.setup

diff --git a/init.d/Makefile b/init.d/Makefile
index 54e5fd5..04e09a7 100644
--- a/init.d/Makefile
+++ b/init.d/Makefile
@@ -1,8 +1,7 @@
 DIR=	${INITDIR}
 SRCS=	bootmisc.in fsck.in hostname.in local.in localmount.in netmount.in \
 	root.in savecache.in swap.in swapfiles.in \
-	tmpfilesd.boot.in tmpfilesd.sysinit.in \
-	swclock.in sysctl.in urandom.in ${SRCS-${OS}}
+	tmpfiles.setup.in swclock.in sysctl.in urandom.in ${SRCS-${OS}}
 BIN=	${OBJS}
 
 # Build our old net foo or not

diff --git a/init.d/tmpfilesd.boot.in b/init.d/tmpfiles.setup.in
similarity index 54%
rename from init.d/tmpfilesd.boot.in
rename to init.d/tmpfiles.setup.in
index 471c128..a3ecd31 100644
--- a/init.d/tmpfilesd.boot.in
+++ b/init.d/tmpfiles.setup.in
@@ -2,7 +2,7 @@
 # Copyright 1999-2012 Gentoo Foundation
 # Released under the 2-clause BSD license.
 
-description="Create tmpfiles.d entries (boot)"
+description="set up tmpfiles.d entries"
 
 depend()
 {
@@ -11,8 +11,8 @@ depend()
 
 start()
 {
-	ebegin "Creating ${description#Create }"
-	@LIBEXECDIR@/sh/tmpfiles.sh --create ${tmpfiles_opts}
+	ebegin "setting up tmpfiles.d entries"
+	@LIBEXECDIR@/sh/tmpfiles.sh --create --remove ${tmpfiles_opts}
 	eend $?
 	return 0
 }

diff --git a/init.d/tmpfilesd.sysinit.in b/init.d/tmpfilesd.sysinit.in
deleted file mode 100644
index 303a1cf..0000000
--- a/init.d/tmpfilesd.sysinit.in
+++ /dev/null
@@ -1,20 +0,0 @@
-#!@PREFIX@/sbin/runscript
-# Copyright 1999-2012 Gentoo Foundation
-# Released under the 2-clause BSD license.
-
-description="Create tmpfiles.d entries (sysinit)"
-
-depend()
-{
-	# Convert to 'need dev' when the new udev is ready, for OpenRC 0.11
-	#need dev-mount
-	use dev
-}
-
-start()
-{
-	ebegin "Creating ${description#Create }"
-	@LIBEXECDIR@/sh/tmpfiles.sh --create ${tmpfiles_opts}
-	eend $?
-	return 0
-}

diff --git a/runlevels/Makefile b/runlevels/Makefile
index a4fd006..e459d15 100644
--- a/runlevels/Makefile
+++ b/runlevels/Makefile
@@ -38,9 +38,9 @@ BOOT-FreeBSD+=	hostid newsyslog savecore syslogd
 # FreeBSD specific stuff
 BOOT-FreeBSD+=	adjkerntz dumpon syscons
 
-BOOT-Linux+=	hwclock keymaps modules mtab procfs termencoding tmpfilesd.boot
+BOOT-Linux+=	hwclock keymaps modules mtab procfs termencoding tmpfiles.setup
 SHUTDOWN-Linux=	killprocs mount-ro
-SYSINIT-Linux=	devfs dmesg sysfs tmpfilesd.sysinit
+SYSINIT-Linux=	devfs dmesg sysfs
 
 # Generic BSD stuff
 BOOT-NetBSD+=	hostid newsyslog savecore syslogd


                 reply	other threads:[~2012-10-31 22:39 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1351723121.c73173ae5389e1a82aaebf2b4503ec7af32052fb.WilliamH@OpenRC \
    --to=williamh@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox