Re: [Buildroot] [PATCH v2] package/openrc: add patch to create /run/lock/subsys

[Arnout Vandecappelle <arnout@mind.be>]

On 09/01/2022 12:05, Norbert Lange wrote:
> Together with changes to sysv/busybox init, this directory
> will already exist on all init systems before any service
> scripts are run.
> 
> Signed-off-by: Norbert Lange <nolange79@gmail.com>

  Applied to master, thanks.

  Normally we require patches to be upstreamed first, but there are already 
plenty of other patches that haven't been sent upstream. It could also be a bit 
buildroot-specific (though I think the subsys is there because some packages 
actually use that directory).

  Still, Michał, do you think this patch would be acceptable upstream? Obviously 
not in its current form since upstream has group uucp rather than daemon.

  Regards,
  Arnout

> ---
> v1->v2:
> *   refresh patch
> ---
>   ...lso-create-run-lock-subsys-directory.patch | 29 +++++++++++++++++++
>   1 file changed, 29 insertions(+)
>   create mode 100644 package/openrc/0006-Also-create-run-lock-subsys-directory.patch
> 
> diff --git a/package/openrc/0006-Also-create-run-lock-subsys-directory.patch b/package/openrc/0006-Also-create-run-lock-subsys-directory.patch
> new file mode 100644
> index 0000000000..cf592a88b0
> --- /dev/null
> +++ b/package/openrc/0006-Also-create-run-lock-subsys-directory.patch
> @@ -0,0 +1,29 @@
> +From 49e5929b468f604d80a0c04a6ce78eb96ca36090 Mon Sep 17 00:00:00 2001
> +From: Norbert Lange <nolange79@gmail.com>
> +Date: Sun, 9 Jan 2022 00:22:40 +0100
> +Subject: [PATCH] Also create /run/lock/subsys directory
> +
> +Buildroot service scripts regularly use this directory, so
> +create it in the most appropriate spot.
> +
> +Signed-off-by: Norbert Lange <nolange79@gmail.com>
> +---
> + sh/init.sh.Linux.in | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/sh/init.sh.Linux.in b/sh/init.sh.Linux.in
> +index e8afc1fb..662ab223 100644
> +--- a/sh/init.sh.Linux.in
> ++++ b/sh/init.sh.Linux.in
> +@@ -82,7 +82,7 @@ elif ! mountinfo -q /run; then
> + fi
> +
> + checkpath -d "$RC_SVCDIR"
> +-checkpath -d -m 0775 -o root:daemon /run/lock
> ++checkpath -d -m 0775 -o root:daemon /run/lock /run/lock/subsys
> +
> + # Try to mount xenfs as early as possible, otherwise rc_sys() will always
> + # return RC_SYS_XENU and will think that we are in a domU while it's not.
> +--
> +2.34.1
> +
> 
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot