All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: [PATCH v4] base-files: add /run directory from FHS 3 draft specification
  2013-05-23  4:02 [PATCH v4] base-files: add /run directory from FHS 3 draft specification Jonathan Liu
@ 2013-05-23  3:52 ` Jonathan Liu
  2013-05-24  3:39 ` Saul Wold
  1 sibling, 0 replies; 4+ messages in thread
From: Jonathan Liu @ 2013-05-23  3:52 UTC (permalink / raw)
  To: openembedded-core

[-- Attachment #1: Type: text/plain, Size: 11914 bytes --]

Note that I have rebased the patch so it depends on:
- [PATCH v4] systemd: update to 204

Regards,
Jonathan


On 23 May 2013 14:02, Jonathan Liu <net147@gmail.com> wrote:

> This adds the /run directory from the Filesystem Hierarchy Standard 3.0
> Draft [1] and refactors the filesystem as follows:
> - Remove creation of /var/volatile/run
> - Remove creation of /var/volatile/lock
> - Remove symbolic link from /var/run to /var/volatile/run
> - Remove symbolic link from /var/lock to /var/volatile/lock
> - Add symbolic link from /var/run to /run
> - Add symbolic link from /var/lock -> /run/lock
> - Add /run to /etc/fstab for sysvinit compatibility
>
> The systemd init script has been modified as follows:
> - Do not create /var/volatile/run
> - Do not create symbolic link from /run to /var/run if /run doesn't
>   exist
>
> A few things have also been cleaned up:
> - Remove udev workaround to ensure /run exists
> - Remove clearing of /var/run as it points to a tmpfs anyway
> - Remove permissions workaround for /var/run/utmp as
>   systemd-update-utmp-runlevel.service starts correctly without it
>
> [1] http://www.linuxbase.org/betaspecs/fhs/fhs.html#runRuntimeVariableData
>
> Signed-off-by: Jonathan Liu <net147@gmail.com>
> ---
>  meta/files/fs-perms.txt                                   |  5 ++---
>  meta/recipes-core/base-files/base-files/fstab             |  1 +
>  meta/recipes-core/base-files/base-files_3.0.14.bb         | 10 +++++-----
>  meta/recipes-core/initscripts/initscripts-1.0/volatiles   |  7 +++----
>  meta/recipes-core/systemd/systemd/00-create-volatile.conf |  2 --
>  meta/recipes-core/systemd/systemd/init                    |  4 ----
>  meta/recipes-core/systemd/systemd/var-run.conf            |  4 ----
>  meta/recipes-core/systemd/systemd_204.bb                  |  3 ---
>  meta/recipes-core/udev/udev.inc                           |  6 +-----
>  meta/recipes-core/udev/udev/init                          |  1 -
>  meta/recipes-core/udev/udev/volatiles                     |  1 -
>  11 files changed, 12 insertions(+), 32 deletions(-)
>  delete mode 100644 meta/recipes-core/systemd/systemd/var-run.conf
>  delete mode 100644 meta/recipes-core/udev/udev/volatiles
>
> diff --git a/meta/files/fs-perms.txt b/meta/files/fs-perms.txt
> index becb85c..c829bec 100644
> --- a/meta/files/fs-perms.txt
> +++ b/meta/files/fs-perms.txt
> @@ -45,15 +45,14 @@ ${oldincludedir}    0755    root    root    true
>  0644    root    root
>
>  # Items from base-files
>  # Links
> -${localstatedir}/run   link    volatile/run
> +${localstatedir}/run   link    /run
>  ${localstatedir}/log   link    volatile/log
> -${localstatedir}/lock  link    volatile/lock
> +${localstatedir}/lock  link    /run/lock
>  ${localstatedir}/tmp   link    volatile/tmp
>
>  # Special permissions from base-files
>  # Set 1777
>  /tmp                           01777   root    root    false - - -
> -${localstatedir}/volatile/lock 01777   root    root    false - - -
>  ${localstatedir}/volatile/tmp  01777   root    root    false - - -
>
>  # Set 2775
> diff --git a/meta/recipes-core/base-files/base-files/fstab
> b/meta/recipes-core/base-files/base-files/fstab
> index f0eed36..dd613a1 100644
> --- a/meta/recipes-core/base-files/base-files/fstab
> +++ b/meta/recipes-core/base-files/base-files/fstab
> @@ -4,6 +4,7 @@ rootfs               /                    auto
> defaults              1  1
>  proc                 /proc                proc       defaults
>  0  0
>  devpts               /dev/pts             devpts     mode=0620,gid=5
>   0  0
>  usbdevfs             /proc/bus/usb        usbdevfs   noauto
>  0  0
> +tmpfs                /run                 tmpfs
>  mode=0755,nodev,nosuid,strictatime 0  0
>  tmpfs                /var/volatile        tmpfs      defaults
>  0  0
>  tmpfs                /media/ram           tmpfs      defaults
>  0  0
>
> diff --git a/meta/recipes-core/base-files/base-files_3.0.14.bbb/meta/recipes-core/base-files/
> base-files_3.0.14.bb
> index a56450d..60185e1 100644
> --- a/meta/recipes-core/base-files/base-files_3.0.14.bb
> +++ b/meta/recipes-core/base-files/base-files_3.0.14.bb
> @@ -31,10 +31,10 @@ S = "${WORKDIR}"
>  INHIBIT_DEFAULT_DEPS = "1"
>
>  docdir_append = "/${P}"
> -dirs1777 = "/tmp ${localstatedir}/volatile/lock
> ${localstatedir}/volatile/tmp"
> +dirs1777 = "/tmp ${localstatedir}/volatile/tmp"
>  dirs2775 = "/home ${prefix}/src ${localstatedir}/local"
>  dirs755 = "/bin /boot /dev ${sysconfdir} ${sysconfdir}/default \
> -           ${sysconfdir}/skel /lib /mnt /proc ${ROOT_HOME} /sbin \
> +           ${sysconfdir}/skel /lib /mnt /proc ${ROOT_HOME} /run /sbin \
>             ${prefix} ${bindir} ${docdir} /usr/games ${includedir} \
>             ${libdir} ${sbindir} ${datadir} \
>             ${datadir}/common-licenses ${datadir}/dict ${infodir} \
> @@ -42,9 +42,7 @@ dirs755 = "/bin /boot /dev ${sysconfdir}
> ${sysconfdir}/default \
>             ${localstatedir}/backups ${localstatedir}/lib \
>             /sys ${localstatedir}/lib/misc ${localstatedir}/spool \
>             ${localstatedir}/volatile \
> -           ${localstatedir}/volatile/lock/subsys \
>             ${localstatedir}/volatile/log \
> -           ${localstatedir}/volatile/run \
>             /mnt /media /media/card /media/cf /media/net /media/ram \
>             /media/union /media/realroot /media/hdd \
>             /media/mmc1"
> @@ -54,7 +52,7 @@ dirs3755 = "/srv  \
>              ${prefix}/local/share ${prefix}/local/src"
>  dirs4775 = "/var/mail"
>
> -volatiles = "run log lock tmp"
> +volatiles = "log tmp"
>  conffiles = "${sysconfdir}/debian_version ${sysconfdir}/host.conf \
>               ${sysconfdir}/inputrc ${sysconfdir}/issue /${sysconfdir}/
> issue.net \
>               ${sysconfdir}/nsswitch.conf ${sysconfdir}/profile \
> @@ -83,6 +81,8 @@ do_install () {
>         for d in card cf net ram; do
>                 ln -sf /media/$d ${D}/mnt/$d
>         done
> +       ln -sf /run ${D}${localstatedir}/run
> +       ln -sf /run/lock ${D}${localstatedir}/lock
>
>         ${BASEFILESISSUEINSTALL}
>
> diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> index 270f128..297245d 100644
> --- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> +++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> @@ -21,13 +21,12 @@
>  # This explains the order of "link before object" as in the example
> above, where
>  # a link will be created at /var/test pointing to /tmp/testfile and due
> to this
>  # link the file defined as /var/test will actually be created as
> /tmp/testfile.
> -d root root 1777 /var/volatile/lock none
> +d root root 1777 /run/lock none
>  d root root 0755 /var/volatile/log none
> -d root root 0755 /var/volatile/run none
>  d root root 1777 /var/volatile/tmp none
> -l root root 1777 /var/lock /var/volatile/lock
> +l root root 1777 /var/lock /run/lock
>  l root root 0755 /var/log /var/volatile/log
> -l root root 0755 /var/run /var/volatile/run
> +l root root 0755 /var/run /run
>  l root root 1777 /var/tmp /var/volatile/tmp
>  l root root 1777 /tmp /var/tmp
>  d root root 0755 /var/lock/subsys none
> diff --git a/meta/recipes-core/systemd/systemd/00-create-volatile.conf
> b/meta/recipes-core/systemd/systemd/00-create-volatile.conf
> index 6163a9c..9ffa88e 100644
> --- a/meta/recipes-core/systemd/systemd/00-create-volatile.conf
> +++ b/meta/recipes-core/systemd/systemd/00-create-volatile.conf
> @@ -4,6 +4,4 @@
>
>
>  d              /var/volatile/log               -       -       -       -
> -d              /var/volatile/lock              -       -       -       -
> -d              /var/volatile/run               -       -       -       -
>  d              /var/volatile/tmp               -       -       -       -
> diff --git a/meta/recipes-core/systemd/systemd/init
> b/meta/recipes-core/systemd/systemd/init
> index ac56cca..7e67a50 100644
> --- a/meta/recipes-core/systemd/systemd/init
> +++ b/meta/recipes-core/systemd/systemd/init
> @@ -42,10 +42,6 @@ case "$1" in
>      [ -e /dev/pts ] || mkdir -m 0755 /dev/pts
>      [ -e /dev/shm ] || mkdir -m 1777 /dev/shm
>      mount -a -t tmpfs 2>/dev/null
> -    mkdir -p /var/volatile/run
> -    if [ ! -e /run ]; then
> -        ln -s /var/run /run
> -    fi
>
>      # cache handling
>      if [ "$DEVCACHE" != "" ]; then
> diff --git a/meta/recipes-core/systemd/systemd/var-run.conf
> b/meta/recipes-core/systemd/systemd/var-run.conf
> deleted file mode 100644
> index c2c18e2..0000000
> --- a/meta/recipes-core/systemd/systemd/var-run.conf
> +++ /dev/null
> @@ -1,4 +0,0 @@
> -# empty /var/run
> -R      /var/run/*      -       -       -       -       -
> -# systemd-update-utmp-runlevel.service fails to start without this
> -f /var/run/utmp 0644 root root - -
> diff --git a/meta/recipes-core/systemd/systemd_204.bbb/meta/recipes-core/systemd/
> systemd_204.bb
> index 8536b2b..74cd3c8 100644
> --- a/meta/recipes-core/systemd/systemd_204.bb
> +++ b/meta/recipes-core/systemd/systemd_204.bb
> @@ -20,7 +20,6 @@ inherit gtk-doc useradd pkgconfig autotools perlnative
> update-rc.d update-altern
>  SRC_URI = "
> http://www.freedesktop.org/software/systemd/systemd-${PV}.tar.xz \
>             file://touchscreen.rules \
>             file://modprobe.rules \
> -           file://var-run.conf \
>             ${UCLIBCPATCHES} \
>
> file://0001-utmp-turn-systemd-update-utmp-shutdown.service-into-.patch \
>             file://00-create-volatile.conf \
> @@ -105,8 +104,6 @@ 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}/00-create-volatile.conf
> ${D}${sysconfdir}/tmpfiles.d/
>
>         if
> ${@base_contains('DISTRO_FEATURES','sysvinit','true','false',d)}; then
> diff --git a/meta/recipes-core/udev/udev.inc
> b/meta/recipes-core/udev/udev.inc
> index 769d861..92485f5 100644
> --- a/meta/recipes-core/udev/udev.inc
> +++ b/meta/recipes-core/udev/udev.inc
> @@ -26,8 +26,7 @@ SRC_URI =
> "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
>             file://local.rules \
>             file://udev-cache \
>             file://udev-cache.default \
> -           file://init \
> -           file://volatiles"
> +           file://init"
>
>  inherit autotools pkgconfig update-rc.d
>
> @@ -85,9 +84,6 @@ do_install_append () {
>         install -d ${D}${sysconfdir}/default
>         install -m 0755 ${WORKDIR}/udev-cache.default
> ${D}${sysconfdir}/default/udev-cache
>
> -       install -d ${D}${sysconfdir}/default/volatiles
> -       install -m 0644 ${WORKDIR}/volatiles
> ${D}${sysconfdir}/default/volatiles/00_udev
> -
>         touch ${D}${sysconfdir}/udev/cache.data
>
>         install -d ${D}${sysconfdir}/udev/rules.d/
> diff --git a/meta/recipes-core/udev/udev/init
> b/meta/recipes-core/udev/udev/init
> index cd87f50..95f9c03 100644
> --- a/meta/recipes-core/udev/udev/init
> +++ b/meta/recipes-core/udev/udev/init
> @@ -50,7 +50,6 @@ case "$1" in
>      [ -e /dev/pts ] || mkdir -m 0755 /dev/pts
>      [ -e /dev/shm ] || mkdir -m 1777 /dev/shm
>      mount -a -t tmpfs 2>/dev/null
> -    mkdir -p /var/volatile/run
>      mkdir -p /var/volatile/tmp
>
>      # cache handling
> diff --git a/meta/recipes-core/udev/udev/volatiles
> b/meta/recipes-core/udev/udev/volatiles
> deleted file mode 100644
> index e33561b..0000000
> --- a/meta/recipes-core/udev/udev/volatiles
> +++ /dev/null
> @@ -1 +0,0 @@
> -l root root 0755 /run /var/run
> --
> 1.8.2.3
>
>

[-- Attachment #2: Type: text/html, Size: 14965 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH v4] base-files: add /run directory from FHS 3 draft specification
@ 2013-05-23  4:02 Jonathan Liu
  2013-05-23  3:52 ` Jonathan Liu
  2013-05-24  3:39 ` Saul Wold
  0 siblings, 2 replies; 4+ messages in thread
From: Jonathan Liu @ 2013-05-23  4:02 UTC (permalink / raw)
  To: openembedded-core

This adds the /run directory from the Filesystem Hierarchy Standard 3.0
Draft [1] and refactors the filesystem as follows:
- Remove creation of /var/volatile/run
- Remove creation of /var/volatile/lock
- Remove symbolic link from /var/run to /var/volatile/run
- Remove symbolic link from /var/lock to /var/volatile/lock
- Add symbolic link from /var/run to /run
- Add symbolic link from /var/lock -> /run/lock
- Add /run to /etc/fstab for sysvinit compatibility

The systemd init script has been modified as follows:
- Do not create /var/volatile/run
- Do not create symbolic link from /run to /var/run if /run doesn't
  exist

A few things have also been cleaned up:
- Remove udev workaround to ensure /run exists
- Remove clearing of /var/run as it points to a tmpfs anyway
- Remove permissions workaround for /var/run/utmp as
  systemd-update-utmp-runlevel.service starts correctly without it

[1] http://www.linuxbase.org/betaspecs/fhs/fhs.html#runRuntimeVariableData

Signed-off-by: Jonathan Liu <net147@gmail.com>
---
 meta/files/fs-perms.txt                                   |  5 ++---
 meta/recipes-core/base-files/base-files/fstab             |  1 +
 meta/recipes-core/base-files/base-files_3.0.14.bb         | 10 +++++-----
 meta/recipes-core/initscripts/initscripts-1.0/volatiles   |  7 +++----
 meta/recipes-core/systemd/systemd/00-create-volatile.conf |  2 --
 meta/recipes-core/systemd/systemd/init                    |  4 ----
 meta/recipes-core/systemd/systemd/var-run.conf            |  4 ----
 meta/recipes-core/systemd/systemd_204.bb                  |  3 ---
 meta/recipes-core/udev/udev.inc                           |  6 +-----
 meta/recipes-core/udev/udev/init                          |  1 -
 meta/recipes-core/udev/udev/volatiles                     |  1 -
 11 files changed, 12 insertions(+), 32 deletions(-)
 delete mode 100644 meta/recipes-core/systemd/systemd/var-run.conf
 delete mode 100644 meta/recipes-core/udev/udev/volatiles

diff --git a/meta/files/fs-perms.txt b/meta/files/fs-perms.txt
index becb85c..c829bec 100644
--- a/meta/files/fs-perms.txt
+++ b/meta/files/fs-perms.txt
@@ -45,15 +45,14 @@ ${oldincludedir}	0755	root	root	true	0644	root	root
 
 # Items from base-files
 # Links
-${localstatedir}/run	link	volatile/run
+${localstatedir}/run	link	/run
 ${localstatedir}/log	link	volatile/log
-${localstatedir}/lock	link	volatile/lock
+${localstatedir}/lock	link	/run/lock
 ${localstatedir}/tmp	link	volatile/tmp
 
 # Special permissions from base-files
 # Set 1777
 /tmp				01777	root	root	false - - -
-${localstatedir}/volatile/lock	01777	root	root	false - - -
 ${localstatedir}/volatile/tmp	01777	root	root	false - - -
 
 # Set 2775
diff --git a/meta/recipes-core/base-files/base-files/fstab b/meta/recipes-core/base-files/base-files/fstab
index f0eed36..dd613a1 100644
--- a/meta/recipes-core/base-files/base-files/fstab
+++ b/meta/recipes-core/base-files/base-files/fstab
@@ -4,6 +4,7 @@ rootfs               /                    auto       defaults              1  1
 proc                 /proc                proc       defaults              0  0
 devpts               /dev/pts             devpts     mode=0620,gid=5       0  0
 usbdevfs             /proc/bus/usb        usbdevfs   noauto                0  0
+tmpfs                /run                 tmpfs      mode=0755,nodev,nosuid,strictatime 0  0
 tmpfs                /var/volatile        tmpfs      defaults              0  0
 tmpfs                /media/ram           tmpfs      defaults              0  0
 
diff --git a/meta/recipes-core/base-files/base-files_3.0.14.bb b/meta/recipes-core/base-files/base-files_3.0.14.bb
index a56450d..60185e1 100644
--- a/meta/recipes-core/base-files/base-files_3.0.14.bb
+++ b/meta/recipes-core/base-files/base-files_3.0.14.bb
@@ -31,10 +31,10 @@ S = "${WORKDIR}"
 INHIBIT_DEFAULT_DEPS = "1"
 
 docdir_append = "/${P}"
-dirs1777 = "/tmp ${localstatedir}/volatile/lock ${localstatedir}/volatile/tmp"
+dirs1777 = "/tmp ${localstatedir}/volatile/tmp"
 dirs2775 = "/home ${prefix}/src ${localstatedir}/local"
 dirs755 = "/bin /boot /dev ${sysconfdir} ${sysconfdir}/default \
-           ${sysconfdir}/skel /lib /mnt /proc ${ROOT_HOME} /sbin \
+           ${sysconfdir}/skel /lib /mnt /proc ${ROOT_HOME} /run /sbin \
            ${prefix} ${bindir} ${docdir} /usr/games ${includedir} \
            ${libdir} ${sbindir} ${datadir} \
            ${datadir}/common-licenses ${datadir}/dict ${infodir} \
@@ -42,9 +42,7 @@ dirs755 = "/bin /boot /dev ${sysconfdir} ${sysconfdir}/default \
            ${localstatedir}/backups ${localstatedir}/lib \
            /sys ${localstatedir}/lib/misc ${localstatedir}/spool \
            ${localstatedir}/volatile \
-           ${localstatedir}/volatile/lock/subsys \
            ${localstatedir}/volatile/log \
-           ${localstatedir}/volatile/run \
            /mnt /media /media/card /media/cf /media/net /media/ram \
            /media/union /media/realroot /media/hdd \
            /media/mmc1"
@@ -54,7 +52,7 @@ dirs3755 = "/srv  \
             ${prefix}/local/share ${prefix}/local/src"
 dirs4775 = "/var/mail"
 
-volatiles = "run log lock tmp"
+volatiles = "log tmp"
 conffiles = "${sysconfdir}/debian_version ${sysconfdir}/host.conf \
              ${sysconfdir}/inputrc ${sysconfdir}/issue /${sysconfdir}/issue.net \
              ${sysconfdir}/nsswitch.conf ${sysconfdir}/profile \
@@ -83,6 +81,8 @@ do_install () {
 	for d in card cf net ram; do
 		ln -sf /media/$d ${D}/mnt/$d
 	done
+	ln -sf /run ${D}${localstatedir}/run
+	ln -sf /run/lock ${D}${localstatedir}/lock
 
 	${BASEFILESISSUEINSTALL}
 
diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
index 270f128..297245d 100644
--- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
+++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
@@ -21,13 +21,12 @@
 # This explains the order of "link before object" as in the example above, where
 # a link will be created at /var/test pointing to /tmp/testfile and due to this
 # link the file defined as /var/test will actually be created as /tmp/testfile.
-d root root 1777 /var/volatile/lock none
+d root root 1777 /run/lock none
 d root root 0755 /var/volatile/log none
-d root root 0755 /var/volatile/run none
 d root root 1777 /var/volatile/tmp none
-l root root 1777 /var/lock /var/volatile/lock
+l root root 1777 /var/lock /run/lock
 l root root 0755 /var/log /var/volatile/log
-l root root 0755 /var/run /var/volatile/run
+l root root 0755 /var/run /run
 l root root 1777 /var/tmp /var/volatile/tmp
 l root root 1777 /tmp /var/tmp
 d root root 0755 /var/lock/subsys none
diff --git a/meta/recipes-core/systemd/systemd/00-create-volatile.conf b/meta/recipes-core/systemd/systemd/00-create-volatile.conf
index 6163a9c..9ffa88e 100644
--- a/meta/recipes-core/systemd/systemd/00-create-volatile.conf
+++ b/meta/recipes-core/systemd/systemd/00-create-volatile.conf
@@ -4,6 +4,4 @@
 
 
 d		/var/volatile/log		-	-	-	-
-d		/var/volatile/lock		-	-	-	-
-d		/var/volatile/run		-	-	-	-
 d		/var/volatile/tmp		-	-	-	-
diff --git a/meta/recipes-core/systemd/systemd/init b/meta/recipes-core/systemd/systemd/init
index ac56cca..7e67a50 100644
--- a/meta/recipes-core/systemd/systemd/init
+++ b/meta/recipes-core/systemd/systemd/init
@@ -42,10 +42,6 @@ case "$1" in
     [ -e /dev/pts ] || mkdir -m 0755 /dev/pts
     [ -e /dev/shm ] || mkdir -m 1777 /dev/shm
     mount -a -t tmpfs 2>/dev/null
-    mkdir -p /var/volatile/run
-    if [ ! -e /run ]; then
-        ln -s /var/run /run
-    fi
 
     # cache handling
     if [ "$DEVCACHE" != "" ]; then
diff --git a/meta/recipes-core/systemd/systemd/var-run.conf b/meta/recipes-core/systemd/systemd/var-run.conf
deleted file mode 100644
index c2c18e2..0000000
--- a/meta/recipes-core/systemd/systemd/var-run.conf
+++ /dev/null
@@ -1,4 +0,0 @@
-# empty /var/run
-R	/var/run/*	-	-	-	-	-
-# systemd-update-utmp-runlevel.service fails to start without this
-f /var/run/utmp 0644 root root - -
diff --git a/meta/recipes-core/systemd/systemd_204.bb b/meta/recipes-core/systemd/systemd_204.bb
index 8536b2b..74cd3c8 100644
--- a/meta/recipes-core/systemd/systemd_204.bb
+++ b/meta/recipes-core/systemd/systemd_204.bb
@@ -20,7 +20,6 @@ inherit gtk-doc useradd pkgconfig autotools perlnative update-rc.d update-altern
 SRC_URI = "http://www.freedesktop.org/software/systemd/systemd-${PV}.tar.xz \
            file://touchscreen.rules \
            file://modprobe.rules \
-           file://var-run.conf \
            ${UCLIBCPATCHES} \
            file://0001-utmp-turn-systemd-update-utmp-shutdown.service-into-.patch \
            file://00-create-volatile.conf \
@@ -105,8 +104,6 @@ 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}/00-create-volatile.conf ${D}${sysconfdir}/tmpfiles.d/
 
 	if ${@base_contains('DISTRO_FEATURES','sysvinit','true','false',d)}; then
diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
index 769d861..92485f5 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -26,8 +26,7 @@ SRC_URI = "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
            file://local.rules \
            file://udev-cache \
            file://udev-cache.default \
-           file://init \
-           file://volatiles"
+           file://init"
 
 inherit autotools pkgconfig update-rc.d
 
@@ -85,9 +84,6 @@ do_install_append () {
 	install -d ${D}${sysconfdir}/default
 	install -m 0755 ${WORKDIR}/udev-cache.default ${D}${sysconfdir}/default/udev-cache
 
-	install -d ${D}${sysconfdir}/default/volatiles
-	install -m 0644 ${WORKDIR}/volatiles ${D}${sysconfdir}/default/volatiles/00_udev
-
 	touch ${D}${sysconfdir}/udev/cache.data
 
 	install -d ${D}${sysconfdir}/udev/rules.d/
diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
index cd87f50..95f9c03 100644
--- a/meta/recipes-core/udev/udev/init
+++ b/meta/recipes-core/udev/udev/init
@@ -50,7 +50,6 @@ case "$1" in
     [ -e /dev/pts ] || mkdir -m 0755 /dev/pts
     [ -e /dev/shm ] || mkdir -m 1777 /dev/shm
     mount -a -t tmpfs 2>/dev/null
-    mkdir -p /var/volatile/run
     mkdir -p /var/volatile/tmp
 
     # cache handling
diff --git a/meta/recipes-core/udev/udev/volatiles b/meta/recipes-core/udev/udev/volatiles
deleted file mode 100644
index e33561b..0000000
--- a/meta/recipes-core/udev/udev/volatiles
+++ /dev/null
@@ -1 +0,0 @@
-l root root 0755 /run /var/run
-- 
1.8.2.3



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v4] base-files: add /run directory from FHS 3 draft specification
  2013-05-23  4:02 [PATCH v4] base-files: add /run directory from FHS 3 draft specification Jonathan Liu
  2013-05-23  3:52 ` Jonathan Liu
@ 2013-05-24  3:39 ` Saul Wold
  2013-05-26 10:56   ` Jonathan Liu
  1 sibling, 1 reply; 4+ messages in thread
From: Saul Wold @ 2013-05-24  3:39 UTC (permalink / raw)
  To: Jonathan Liu; +Cc: openembedded-core

On 05/22/2013 09:02 PM, Jonathan Liu wrote:
> This adds the /run directory from the Filesystem Hierarchy Standard 3.0
> Draft [1] and refactors the filesystem as follows:
> - Remove creation of /var/volatile/run
> - Remove creation of /var/volatile/lock
> - Remove symbolic link from /var/run to /var/volatile/run
> - Remove symbolic link from /var/lock to /var/volatile/lock
> - Add symbolic link from /var/run to /run
> - Add symbolic link from /var/lock -> /run/lock
> - Add /run to /etc/fstab for sysvinit compatibility
>
> The systemd init script has been modified as follows:
> - Do not create /var/volatile/run
> - Do not create symbolic link from /run to /var/run if /run doesn't
>    exist
>
> A few things have also been cleaned up:
> - Remove udev workaround to ensure /run exists
> - Remove clearing of /var/run as it points to a tmpfs anyway
> - Remove permissions workaround for /var/run/utmp as
>    systemd-update-utmp-runlevel.service starts correctly without it
>
> [1] http://www.linuxbase.org/betaspecs/fhs/fhs.html#runRuntimeVariableData
>

Not quite done yet, Ross requested that you break this into smaller 
patches since this contains un-related changes, their kind of related 
but seperatable.

I also noticed a few installed not shipped warnings pop up for 
consolekit, bind and cups, all having to do with /run, these may need 
additional patching.

Please build a world with this changes to possibly see these warnings.

Sau!


> Signed-off-by: Jonathan Liu <net147@gmail.com>
> ---
>   meta/files/fs-perms.txt                                   |  5 ++---
>   meta/recipes-core/base-files/base-files/fstab             |  1 +
>   meta/recipes-core/base-files/base-files_3.0.14.bb         | 10 +++++-----
>   meta/recipes-core/initscripts/initscripts-1.0/volatiles   |  7 +++----
>   meta/recipes-core/systemd/systemd/00-create-volatile.conf |  2 --
>   meta/recipes-core/systemd/systemd/init                    |  4 ----
>   meta/recipes-core/systemd/systemd/var-run.conf            |  4 ----
>   meta/recipes-core/systemd/systemd_204.bb                  |  3 ---
>   meta/recipes-core/udev/udev.inc                           |  6 +-----
>   meta/recipes-core/udev/udev/init                          |  1 -
>   meta/recipes-core/udev/udev/volatiles                     |  1 -
>   11 files changed, 12 insertions(+), 32 deletions(-)
>   delete mode 100644 meta/recipes-core/systemd/systemd/var-run.conf
>   delete mode 100644 meta/recipes-core/udev/udev/volatiles
>
> diff --git a/meta/files/fs-perms.txt b/meta/files/fs-perms.txt
> index becb85c..c829bec 100644
> --- a/meta/files/fs-perms.txt
> +++ b/meta/files/fs-perms.txt
> @@ -45,15 +45,14 @@ ${oldincludedir}	0755	root	root	true	0644	root	root
>
>   # Items from base-files
>   # Links
> -${localstatedir}/run	link	volatile/run
> +${localstatedir}/run	link	/run
>   ${localstatedir}/log	link	volatile/log
> -${localstatedir}/lock	link	volatile/lock
> +${localstatedir}/lock	link	/run/lock
>   ${localstatedir}/tmp	link	volatile/tmp
>
>   # Special permissions from base-files
>   # Set 1777
>   /tmp				01777	root	root	false - - -
> -${localstatedir}/volatile/lock	01777	root	root	false - - -
>   ${localstatedir}/volatile/tmp	01777	root	root	false - - -
>
>   # Set 2775
> diff --git a/meta/recipes-core/base-files/base-files/fstab b/meta/recipes-core/base-files/base-files/fstab
> index f0eed36..dd613a1 100644
> --- a/meta/recipes-core/base-files/base-files/fstab
> +++ b/meta/recipes-core/base-files/base-files/fstab
> @@ -4,6 +4,7 @@ rootfs               /                    auto       defaults              1  1
>   proc                 /proc                proc       defaults              0  0
>   devpts               /dev/pts             devpts     mode=0620,gid=5       0  0
>   usbdevfs             /proc/bus/usb        usbdevfs   noauto                0  0
> +tmpfs                /run                 tmpfs      mode=0755,nodev,nosuid,strictatime 0  0
>   tmpfs                /var/volatile        tmpfs      defaults              0  0
>   tmpfs                /media/ram           tmpfs      defaults              0  0
>
> diff --git a/meta/recipes-core/base-files/base-files_3.0.14.bb b/meta/recipes-core/base-files/base-files_3.0.14.bb
> index a56450d..60185e1 100644
> --- a/meta/recipes-core/base-files/base-files_3.0.14.bb
> +++ b/meta/recipes-core/base-files/base-files_3.0.14.bb
> @@ -31,10 +31,10 @@ S = "${WORKDIR}"
>   INHIBIT_DEFAULT_DEPS = "1"
>
>   docdir_append = "/${P}"
> -dirs1777 = "/tmp ${localstatedir}/volatile/lock ${localstatedir}/volatile/tmp"
> +dirs1777 = "/tmp ${localstatedir}/volatile/tmp"
>   dirs2775 = "/home ${prefix}/src ${localstatedir}/local"
>   dirs755 = "/bin /boot /dev ${sysconfdir} ${sysconfdir}/default \
> -           ${sysconfdir}/skel /lib /mnt /proc ${ROOT_HOME} /sbin \
> +           ${sysconfdir}/skel /lib /mnt /proc ${ROOT_HOME} /run /sbin \
>              ${prefix} ${bindir} ${docdir} /usr/games ${includedir} \
>              ${libdir} ${sbindir} ${datadir} \
>              ${datadir}/common-licenses ${datadir}/dict ${infodir} \
> @@ -42,9 +42,7 @@ dirs755 = "/bin /boot /dev ${sysconfdir} ${sysconfdir}/default \
>              ${localstatedir}/backups ${localstatedir}/lib \
>              /sys ${localstatedir}/lib/misc ${localstatedir}/spool \
>              ${localstatedir}/volatile \
> -           ${localstatedir}/volatile/lock/subsys \
>              ${localstatedir}/volatile/log \
> -           ${localstatedir}/volatile/run \
>              /mnt /media /media/card /media/cf /media/net /media/ram \
>              /media/union /media/realroot /media/hdd \
>              /media/mmc1"
> @@ -54,7 +52,7 @@ dirs3755 = "/srv  \
>               ${prefix}/local/share ${prefix}/local/src"
>   dirs4775 = "/var/mail"
>
> -volatiles = "run log lock tmp"
> +volatiles = "log tmp"
>   conffiles = "${sysconfdir}/debian_version ${sysconfdir}/host.conf \
>                ${sysconfdir}/inputrc ${sysconfdir}/issue /${sysconfdir}/issue.net \
>                ${sysconfdir}/nsswitch.conf ${sysconfdir}/profile \
> @@ -83,6 +81,8 @@ do_install () {
>   	for d in card cf net ram; do
>   		ln -sf /media/$d ${D}/mnt/$d
>   	done
> +	ln -sf /run ${D}${localstatedir}/run
> +	ln -sf /run/lock ${D}${localstatedir}/lock
>
>   	${BASEFILESISSUEINSTALL}
>
> diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> index 270f128..297245d 100644
> --- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> +++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
> @@ -21,13 +21,12 @@
>   # This explains the order of "link before object" as in the example above, where
>   # a link will be created at /var/test pointing to /tmp/testfile and due to this
>   # link the file defined as /var/test will actually be created as /tmp/testfile.
> -d root root 1777 /var/volatile/lock none
> +d root root 1777 /run/lock none
>   d root root 0755 /var/volatile/log none
> -d root root 0755 /var/volatile/run none
>   d root root 1777 /var/volatile/tmp none
> -l root root 1777 /var/lock /var/volatile/lock
> +l root root 1777 /var/lock /run/lock
>   l root root 0755 /var/log /var/volatile/log
> -l root root 0755 /var/run /var/volatile/run
> +l root root 0755 /var/run /run
>   l root root 1777 /var/tmp /var/volatile/tmp
>   l root root 1777 /tmp /var/tmp
>   d root root 0755 /var/lock/subsys none
> diff --git a/meta/recipes-core/systemd/systemd/00-create-volatile.conf b/meta/recipes-core/systemd/systemd/00-create-volatile.conf
> index 6163a9c..9ffa88e 100644
> --- a/meta/recipes-core/systemd/systemd/00-create-volatile.conf
> +++ b/meta/recipes-core/systemd/systemd/00-create-volatile.conf
> @@ -4,6 +4,4 @@
>
>
>   d		/var/volatile/log		-	-	-	-
> -d		/var/volatile/lock		-	-	-	-
> -d		/var/volatile/run		-	-	-	-
>   d		/var/volatile/tmp		-	-	-	-
> diff --git a/meta/recipes-core/systemd/systemd/init b/meta/recipes-core/systemd/systemd/init
> index ac56cca..7e67a50 100644
> --- a/meta/recipes-core/systemd/systemd/init
> +++ b/meta/recipes-core/systemd/systemd/init
> @@ -42,10 +42,6 @@ case "$1" in
>       [ -e /dev/pts ] || mkdir -m 0755 /dev/pts
>       [ -e /dev/shm ] || mkdir -m 1777 /dev/shm
>       mount -a -t tmpfs 2>/dev/null
> -    mkdir -p /var/volatile/run
> -    if [ ! -e /run ]; then
> -        ln -s /var/run /run
> -    fi
>
>       # cache handling
>       if [ "$DEVCACHE" != "" ]; then
> diff --git a/meta/recipes-core/systemd/systemd/var-run.conf b/meta/recipes-core/systemd/systemd/var-run.conf
> deleted file mode 100644
> index c2c18e2..0000000
> --- a/meta/recipes-core/systemd/systemd/var-run.conf
> +++ /dev/null
> @@ -1,4 +0,0 @@
> -# empty /var/run
> -R	/var/run/*	-	-	-	-	-
> -# systemd-update-utmp-runlevel.service fails to start without this
> -f /var/run/utmp 0644 root root - -
> diff --git a/meta/recipes-core/systemd/systemd_204.bb b/meta/recipes-core/systemd/systemd_204.bb
> index 8536b2b..74cd3c8 100644
> --- a/meta/recipes-core/systemd/systemd_204.bb
> +++ b/meta/recipes-core/systemd/systemd_204.bb
> @@ -20,7 +20,6 @@ inherit gtk-doc useradd pkgconfig autotools perlnative update-rc.d update-altern
>   SRC_URI = "http://www.freedesktop.org/software/systemd/systemd-${PV}.tar.xz \
>              file://touchscreen.rules \
>              file://modprobe.rules \
> -           file://var-run.conf \
>              ${UCLIBCPATCHES} \
>              file://0001-utmp-turn-systemd-update-utmp-shutdown.service-into-.patch \
>              file://00-create-volatile.conf \
> @@ -105,8 +104,6 @@ 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}/00-create-volatile.conf ${D}${sysconfdir}/tmpfiles.d/
>
>   	if ${@base_contains('DISTRO_FEATURES','sysvinit','true','false',d)}; then
> diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
> index 769d861..92485f5 100644
> --- a/meta/recipes-core/udev/udev.inc
> +++ b/meta/recipes-core/udev/udev.inc
> @@ -26,8 +26,7 @@ SRC_URI = "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
>              file://local.rules \
>              file://udev-cache \
>              file://udev-cache.default \
> -           file://init \
> -           file://volatiles"
> +           file://init"
>
>   inherit autotools pkgconfig update-rc.d
>
> @@ -85,9 +84,6 @@ do_install_append () {
>   	install -d ${D}${sysconfdir}/default
>   	install -m 0755 ${WORKDIR}/udev-cache.default ${D}${sysconfdir}/default/udev-cache
>
> -	install -d ${D}${sysconfdir}/default/volatiles
> -	install -m 0644 ${WORKDIR}/volatiles ${D}${sysconfdir}/default/volatiles/00_udev
> -
>   	touch ${D}${sysconfdir}/udev/cache.data
>
>   	install -d ${D}${sysconfdir}/udev/rules.d/
> diff --git a/meta/recipes-core/udev/udev/init b/meta/recipes-core/udev/udev/init
> index cd87f50..95f9c03 100644
> --- a/meta/recipes-core/udev/udev/init
> +++ b/meta/recipes-core/udev/udev/init
> @@ -50,7 +50,6 @@ case "$1" in
>       [ -e /dev/pts ] || mkdir -m 0755 /dev/pts
>       [ -e /dev/shm ] || mkdir -m 1777 /dev/shm
>       mount -a -t tmpfs 2>/dev/null
> -    mkdir -p /var/volatile/run
>       mkdir -p /var/volatile/tmp
>
>       # cache handling
> diff --git a/meta/recipes-core/udev/udev/volatiles b/meta/recipes-core/udev/udev/volatiles
> deleted file mode 100644
> index e33561b..0000000
> --- a/meta/recipes-core/udev/udev/volatiles
> +++ /dev/null
> @@ -1 +0,0 @@
> -l root root 0755 /run /var/run
>


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v4] base-files: add /run directory from FHS 3 draft specification
  2013-05-24  3:39 ` Saul Wold
@ 2013-05-26 10:56   ` Jonathan Liu
  0 siblings, 0 replies; 4+ messages in thread
From: Jonathan Liu @ 2013-05-26 10:56 UTC (permalink / raw)
  To: Saul Wold; +Cc: openembedded-core

On 24/05/2013 1:39 PM, Saul Wold wrote:
> On 05/22/2013 09:02 PM, Jonathan Liu wrote:
>> This adds the /run directory from the Filesystem Hierarchy Standard 3.0
>> Draft [1] and refactors the filesystem as follows:
>> - Remove creation of /var/volatile/run
>> - Remove creation of /var/volatile/lock
>> - Remove symbolic link from /var/run to /var/volatile/run
>> - Remove symbolic link from /var/lock to /var/volatile/lock
>> - Add symbolic link from /var/run to /run
>> - Add symbolic link from /var/lock -> /run/lock
>> - Add /run to /etc/fstab for sysvinit compatibility
>>
>> The systemd init script has been modified as follows:
>> - Do not create /var/volatile/run
>> - Do not create symbolic link from /run to /var/run if /run doesn't
>>    exist
>>
>> A few things have also been cleaned up:
>> - Remove udev workaround to ensure /run exists
>> - Remove clearing of /var/run as it points to a tmpfs anyway
>> - Remove permissions workaround for /var/run/utmp as
>>    systemd-update-utmp-runlevel.service starts correctly without it
>>
>> [1] 
>> http://www.linuxbase.org/betaspecs/fhs/fhs.html#runRuntimeVariableData
>>
>
> Not quite done yet, Ross requested that you break this into smaller 
> patches since this contains un-related changes, their kind of related 
> but seperatable.
>
> I also noticed a few installed not shipped warnings pop up for 
> consolekit, bind and cups, all having to do with /run, these may need 
> additional patching.
>
> Please build a world with this changes to possibly see these warnings.
>
> Sau!
All done.

Regards,
Jonathan
>
>
>> Signed-off-by: Jonathan Liu <net147@gmail.com>
>> ---
>>   meta/files/fs-perms.txt                                   |  5 ++---
>>   meta/recipes-core/base-files/base-files/fstab             |  1 +
>>   meta/recipes-core/base-files/base-files_3.0.14.bb         | 10 
>> +++++-----
>>   meta/recipes-core/initscripts/initscripts-1.0/volatiles   |  7 +++----
>>   meta/recipes-core/systemd/systemd/00-create-volatile.conf |  2 --
>>   meta/recipes-core/systemd/systemd/init                    |  4 ----
>>   meta/recipes-core/systemd/systemd/var-run.conf            |  4 ----
>>   meta/recipes-core/systemd/systemd_204.bb                  |  3 ---
>>   meta/recipes-core/udev/udev.inc                           |  6 +-----
>>   meta/recipes-core/udev/udev/init                          |  1 -
>>   meta/recipes-core/udev/udev/volatiles                     |  1 -
>>   11 files changed, 12 insertions(+), 32 deletions(-)
>>   delete mode 100644 meta/recipes-core/systemd/systemd/var-run.conf
>>   delete mode 100644 meta/recipes-core/udev/udev/volatiles
>>
>> diff --git a/meta/files/fs-perms.txt b/meta/files/fs-perms.txt
>> index becb85c..c829bec 100644
>> --- a/meta/files/fs-perms.txt
>> +++ b/meta/files/fs-perms.txt
>> @@ -45,15 +45,14 @@ ${oldincludedir}    0755    root    root true    
>> 0644    root    root
>>
>>   # Items from base-files
>>   # Links
>> -${localstatedir}/run    link    volatile/run
>> +${localstatedir}/run    link    /run
>>   ${localstatedir}/log    link    volatile/log
>> -${localstatedir}/lock    link    volatile/lock
>> +${localstatedir}/lock    link    /run/lock
>>   ${localstatedir}/tmp    link    volatile/tmp
>>
>>   # Special permissions from base-files
>>   # Set 1777
>>   /tmp                01777    root    root    false - - -
>> -${localstatedir}/volatile/lock    01777    root    root false - - -
>>   ${localstatedir}/volatile/tmp    01777    root    root false - - -
>>
>>   # Set 2775
>> diff --git a/meta/recipes-core/base-files/base-files/fstab 
>> b/meta/recipes-core/base-files/base-files/fstab
>> index f0eed36..dd613a1 100644
>> --- a/meta/recipes-core/base-files/base-files/fstab
>> +++ b/meta/recipes-core/base-files/base-files/fstab
>> @@ -4,6 +4,7 @@ rootfs               / auto       
>> defaults              1  1
>>   proc                 /proc                proc 
>> defaults              0  0
>>   devpts               /dev/pts             devpts 
>> mode=0620,gid=5       0  0
>>   usbdevfs             /proc/bus/usb        usbdevfs 
>> noauto                0  0
>> +tmpfs                /run                 tmpfs 
>> mode=0755,nodev,nosuid,strictatime 0  0
>>   tmpfs                /var/volatile        tmpfs 
>> defaults              0  0
>>   tmpfs                /media/ram           tmpfs 
>> defaults              0  0
>>
>> diff --git a/meta/recipes-core/base-files/base-files_3.0.14.bb 
>> b/meta/recipes-core/base-files/base-files_3.0.14.bb
>> index a56450d..60185e1 100644
>> --- a/meta/recipes-core/base-files/base-files_3.0.14.bb
>> +++ b/meta/recipes-core/base-files/base-files_3.0.14.bb
>> @@ -31,10 +31,10 @@ S = "${WORKDIR}"
>>   INHIBIT_DEFAULT_DEPS = "1"
>>
>>   docdir_append = "/${P}"
>> -dirs1777 = "/tmp ${localstatedir}/volatile/lock 
>> ${localstatedir}/volatile/tmp"
>> +dirs1777 = "/tmp ${localstatedir}/volatile/tmp"
>>   dirs2775 = "/home ${prefix}/src ${localstatedir}/local"
>>   dirs755 = "/bin /boot /dev ${sysconfdir} ${sysconfdir}/default \
>> -           ${sysconfdir}/skel /lib /mnt /proc ${ROOT_HOME} /sbin \
>> +           ${sysconfdir}/skel /lib /mnt /proc ${ROOT_HOME} /run /sbin \
>>              ${prefix} ${bindir} ${docdir} /usr/games ${includedir} \
>>              ${libdir} ${sbindir} ${datadir} \
>>              ${datadir}/common-licenses ${datadir}/dict ${infodir} \
>> @@ -42,9 +42,7 @@ dirs755 = "/bin /boot /dev ${sysconfdir} 
>> ${sysconfdir}/default \
>>              ${localstatedir}/backups ${localstatedir}/lib \
>>              /sys ${localstatedir}/lib/misc ${localstatedir}/spool \
>>              ${localstatedir}/volatile \
>> -           ${localstatedir}/volatile/lock/subsys \
>>              ${localstatedir}/volatile/log \
>> -           ${localstatedir}/volatile/run \
>>              /mnt /media /media/card /media/cf /media/net /media/ram \
>>              /media/union /media/realroot /media/hdd \
>>              /media/mmc1"
>> @@ -54,7 +52,7 @@ dirs3755 = "/srv  \
>>               ${prefix}/local/share ${prefix}/local/src"
>>   dirs4775 = "/var/mail"
>>
>> -volatiles = "run log lock tmp"
>> +volatiles = "log tmp"
>>   conffiles = "${sysconfdir}/debian_version ${sysconfdir}/host.conf \
>>                ${sysconfdir}/inputrc ${sysconfdir}/issue 
>> /${sysconfdir}/issue.net \
>>                ${sysconfdir}/nsswitch.conf ${sysconfdir}/profile \
>> @@ -83,6 +81,8 @@ do_install () {
>>       for d in card cf net ram; do
>>           ln -sf /media/$d ${D}/mnt/$d
>>       done
>> +    ln -sf /run ${D}${localstatedir}/run
>> +    ln -sf /run/lock ${D}${localstatedir}/lock
>>
>>       ${BASEFILESISSUEINSTALL}
>>
>> diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles 
>> b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
>> index 270f128..297245d 100644
>> --- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
>> +++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
>> @@ -21,13 +21,12 @@
>>   # This explains the order of "link before object" as in the example 
>> above, where
>>   # a link will be created at /var/test pointing to /tmp/testfile and 
>> due to this
>>   # link the file defined as /var/test will actually be created as 
>> /tmp/testfile.
>> -d root root 1777 /var/volatile/lock none
>> +d root root 1777 /run/lock none
>>   d root root 0755 /var/volatile/log none
>> -d root root 0755 /var/volatile/run none
>>   d root root 1777 /var/volatile/tmp none
>> -l root root 1777 /var/lock /var/volatile/lock
>> +l root root 1777 /var/lock /run/lock
>>   l root root 0755 /var/log /var/volatile/log
>> -l root root 0755 /var/run /var/volatile/run
>> +l root root 0755 /var/run /run
>>   l root root 1777 /var/tmp /var/volatile/tmp
>>   l root root 1777 /tmp /var/tmp
>>   d root root 0755 /var/lock/subsys none
>> diff --git 
>> a/meta/recipes-core/systemd/systemd/00-create-volatile.conf 
>> b/meta/recipes-core/systemd/systemd/00-create-volatile.conf
>> index 6163a9c..9ffa88e 100644
>> --- a/meta/recipes-core/systemd/systemd/00-create-volatile.conf
>> +++ b/meta/recipes-core/systemd/systemd/00-create-volatile.conf
>> @@ -4,6 +4,4 @@
>>
>>
>>   d        /var/volatile/log        -    -    -    -
>> -d        /var/volatile/lock        -    -    -    -
>> -d        /var/volatile/run        -    -    -    -
>>   d        /var/volatile/tmp        -    -    -    -
>> diff --git a/meta/recipes-core/systemd/systemd/init 
>> b/meta/recipes-core/systemd/systemd/init
>> index ac56cca..7e67a50 100644
>> --- a/meta/recipes-core/systemd/systemd/init
>> +++ b/meta/recipes-core/systemd/systemd/init
>> @@ -42,10 +42,6 @@ case "$1" in
>>       [ -e /dev/pts ] || mkdir -m 0755 /dev/pts
>>       [ -e /dev/shm ] || mkdir -m 1777 /dev/shm
>>       mount -a -t tmpfs 2>/dev/null
>> -    mkdir -p /var/volatile/run
>> -    if [ ! -e /run ]; then
>> -        ln -s /var/run /run
>> -    fi
>>
>>       # cache handling
>>       if [ "$DEVCACHE" != "" ]; then
>> diff --git a/meta/recipes-core/systemd/systemd/var-run.conf 
>> b/meta/recipes-core/systemd/systemd/var-run.conf
>> deleted file mode 100644
>> index c2c18e2..0000000
>> --- a/meta/recipes-core/systemd/systemd/var-run.conf
>> +++ /dev/null
>> @@ -1,4 +0,0 @@
>> -# empty /var/run
>> -R    /var/run/*    -    -    -    -    -
>> -# systemd-update-utmp-runlevel.service fails to start without this
>> -f /var/run/utmp 0644 root root - -
>> diff --git a/meta/recipes-core/systemd/systemd_204.bb 
>> b/meta/recipes-core/systemd/systemd_204.bb
>> index 8536b2b..74cd3c8 100644
>> --- a/meta/recipes-core/systemd/systemd_204.bb
>> +++ b/meta/recipes-core/systemd/systemd_204.bb
>> @@ -20,7 +20,6 @@ inherit gtk-doc useradd pkgconfig autotools 
>> perlnative update-rc.d update-altern
>>   SRC_URI = 
>> "http://www.freedesktop.org/software/systemd/systemd-${PV}.tar.xz \
>>              file://touchscreen.rules \
>>              file://modprobe.rules \
>> -           file://var-run.conf \
>>              ${UCLIBCPATCHES} \
>> file://0001-utmp-turn-systemd-update-utmp-shutdown.service-into-.patch \
>>              file://00-create-volatile.conf \
>> @@ -105,8 +104,6 @@ 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}/00-create-volatile.conf 
>> ${D}${sysconfdir}/tmpfiles.d/
>>
>>       if 
>> ${@base_contains('DISTRO_FEATURES','sysvinit','true','false',d)}; then
>> diff --git a/meta/recipes-core/udev/udev.inc 
>> b/meta/recipes-core/udev/udev.inc
>> index 769d861..92485f5 100644
>> --- a/meta/recipes-core/udev/udev.inc
>> +++ b/meta/recipes-core/udev/udev.inc
>> @@ -26,8 +26,7 @@ SRC_URI = 
>> "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
>>              file://local.rules \
>>              file://udev-cache \
>>              file://udev-cache.default \
>> -           file://init \
>> -           file://volatiles"
>> +           file://init"
>>
>>   inherit autotools pkgconfig update-rc.d
>>
>> @@ -85,9 +84,6 @@ do_install_append () {
>>       install -d ${D}${sysconfdir}/default
>>       install -m 0755 ${WORKDIR}/udev-cache.default 
>> ${D}${sysconfdir}/default/udev-cache
>>
>> -    install -d ${D}${sysconfdir}/default/volatiles
>> -    install -m 0644 ${WORKDIR}/volatiles 
>> ${D}${sysconfdir}/default/volatiles/00_udev
>> -
>>       touch ${D}${sysconfdir}/udev/cache.data
>>
>>       install -d ${D}${sysconfdir}/udev/rules.d/
>> diff --git a/meta/recipes-core/udev/udev/init 
>> b/meta/recipes-core/udev/udev/init
>> index cd87f50..95f9c03 100644
>> --- a/meta/recipes-core/udev/udev/init
>> +++ b/meta/recipes-core/udev/udev/init
>> @@ -50,7 +50,6 @@ case "$1" in
>>       [ -e /dev/pts ] || mkdir -m 0755 /dev/pts
>>       [ -e /dev/shm ] || mkdir -m 1777 /dev/shm
>>       mount -a -t tmpfs 2>/dev/null
>> -    mkdir -p /var/volatile/run
>>       mkdir -p /var/volatile/tmp
>>
>>       # cache handling
>> diff --git a/meta/recipes-core/udev/udev/volatiles 
>> b/meta/recipes-core/udev/udev/volatiles
>> deleted file mode 100644
>> index e33561b..0000000
>> --- a/meta/recipes-core/udev/udev/volatiles
>> +++ /dev/null
>> @@ -1 +0,0 @@
>> -l root root 0755 /run /var/run
>>



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-05-26 11:00 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-05-23  4:02 [PATCH v4] base-files: add /run directory from FHS 3 draft specification Jonathan Liu
2013-05-23  3:52 ` Jonathan Liu
2013-05-24  3:39 ` Saul Wold
2013-05-26 10:56   ` Jonathan Liu

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.