All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Peter A. Bigot" <pab@pabigot.com>
To: Holger Freyther <holger+oe@freyther.de>
Cc: openembedded-devel@lists.openembedded.org,
	Holger Hans Peter Freyther <zecke@selfish.org>
Subject: Re: meta-systemd: Introduce a create-volatile to create tmp directories
Date: Tue, 22 Jan 2013 09:02:12 -0600	[thread overview]
Message-ID: <50FEA9F4.4080508@pabigot.com> (raw)
In-Reply-To: <1347641974-4993-1-git-send-email-holger+oe@freyther.de>

On 09/14/2012 11:59 AM, Holger Freyther wrote:
> From: Holger Hans Peter Freyther <zecke@selfish.org>
>
> The base-files create symlinks from /var/{log,tmp,lock,cache,run} to
> the /var/volatile. The directories are created by sysv initscripts.
> Use the systemd-tmpfiles to create the directories. One needs to be
> careful with the naming of the file as systemd first collects all
> *.conf files, sorts them by name and is parsing them then. Make sure
> the new file is parsed before the systemd.conf.

(This is http://patches.openembedded.org/patch/36551/ which has been 
archived, but AFAIK I can't reply to the full context because I was 
unsubscribed when the discussion occurred:)

Apologies if my recursive grep skills have failed, but it appears this 
is still not globally resolved for images that use systemd as the init 
manager.

meta-gumstix has this patch and it caused problems with attempts to use 
systemd_v189 in denzil since it's hidden inside a bbappend to 
systemd_git.  So I'm hoping to identify a more forward-looking 
solution.  Is there a plan to fix this issue globally?  Koen preferred 
using base_files+fs_perms as in Angstrom, but Holger was concerned this 
would prevent sharing base_files between sysvinit and systemd builds.

Peter

>
> ---
> .../systemd/systemd/create-volatile.conf           |   11 +++++++++++
>   meta-systemd/recipes-core/systemd/systemd_git.bb   |    4 +++-
>   2 files changed, 14 insertions(+), 1 deletion(-)
>   create mode 100644 meta-systemd/recipes-core/systemd/systemd/create-volatile.conf
>
> diff --git a/meta-systemd/recipes-core/systemd/systemd/create-volatile.conf b/meta-systemd/recipes-core/systemd/systemd/create-volatile.conf
> new file mode 100644
> index 0000000..afcf8c3
> --- /dev/null
> +++ b/meta-systemd/recipes-core/systemd/systemd/create-volatile.conf
> @@ -0,0 +1,11 @@
> +# This goes hand-in-hand with the base-files of OE-Core. The file must
> +# be sorted before 'systemd.conf' becuase this attempts to create a file
> +# inside /var/log.
> +
> +
> +d   /var/volatile/cache -   -   -   -
> +d   /var/volatile/log   -   -   -   -
> +d   /var/volatile/lock  -   -   -   -
> +d   /var/volatile/run   -   -   -   -
> +d   /var/volatile/tmp   -   -   -   -
> +
> diff --git a/meta-systemd/recipes-core/systemd/systemd_git.bb b/meta-systemd/recipes-core/systemd/systemd_git.bb
> index 09bc635..2b245c0 100644
> --- a/meta-systemd/recipes-core/systemd/systemd_git.bb
> +++ b/meta-systemd/recipes-core/systemd/systemd_git.bb
> @@ -19,7 +19,7 @@ inherit gitpkgv
>   PKGV = "v${GITPKGVTAG}"
>   
>   PV = "git"
> -PR = "r6"
> +PR = "r7"
>   
>   inherit useradd pkgconfig autotools perlnative
>   
> @@ -31,6 +31,7 @@ SRC_URI = "git://anongit.freedesktop.org/systemd/systemd;protocol=git \
>              file://touchscreen.rules \
>              file://modprobe.rules \
>              file://var-run.conf \
> +           file://create-volatile.conf \
>              ${UCLIBCPATCHES} \
>             "
>   UCLIBCPATCHES = ""
> @@ -97,6 +98,7 @@ do_install() {
>   	install -m 0644 ${WORKDIR}/*.rules ${D}${sysconfdir}/udev/rules.d/
>   
>   	install -m 0644 ${WORKDIR}/var-run.conf ${D}${sysconfdir}/tmpfiles.d/
> +	install -m 0644 ${WORKDIR}/create-volatile.conf ${D}${sysconfdir}/tmpfiles.d/
>   }
>   
>   python populate_packages_prepend (){




  parent reply	other threads:[~2013-01-22 15:20 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-14 16:59 [PATCH] meta-systemd: Introduce a create-volatile to create tmp directories Holger Hans Peter Freyther
2012-09-15  9:26 ` Koen Kooi
2012-09-15 13:20   ` Holger Hans Peter Freyther
2013-01-22 15:02 ` Peter A. Bigot [this message]
     [not found]   ` <20130123195059.GG2634@xiaoyu.lan>
2013-01-23 20:44     ` Peter A. Bigot

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=50FEA9F4.4080508@pabigot.com \
    --to=pab@pabigot.com \
    --cc=holger+oe@freyther.de \
    --cc=openembedded-devel@lists.openembedded.org \
    --cc=zecke@selfish.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.