libpam: use /run instead of /var/run in systemd tmpfiles

Message ID 20220317030519.1423570-1-ricardo@foundries.io
State Accepted, archived
Commit 09eabeff2168c416c18b1c375e095b472830a9b0
Headers show
Series libpam: use /run instead of /var/run in systemd tmpfiles | expand

Commit Message

Ricardo Salveti March 17, 2022, 3:05 a.m. UTC
/var/run in deprecated by systemd, use /run instead, as suggested by systemd.

Signed-off-by: Ricardo Salveti <ricardo@foundries.io>
---
 meta/recipes-extended/pam/libpam/pam-volatiles.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alexander Kanavin March 17, 2022, 6:53 a.m. UTC | #1
A link to back up the claim would be helpful. Also, what makes this file
and setting systemd specific?

Alex

On Thu 17. Mar 2022 at 4.05, Ricardo Salveti <ricardo@foundries.io> wrote:

> /var/run in deprecated by systemd, use /run instead, as suggested by
> systemd.
>
> Signed-off-by: Ricardo Salveti <ricardo@foundries.io>
> ---
>  meta/recipes-extended/pam/libpam/pam-volatiles.conf | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-extended/pam/libpam/pam-volatiles.conf
> b/meta/recipes-extended/pam/libpam/pam-volatiles.conf
> index d8b8259a93..1263feb03d 100644
> --- a/meta/recipes-extended/pam/libpam/pam-volatiles.conf
> +++ b/meta/recipes-extended/pam/libpam/pam-volatiles.conf
> @@ -1 +1 @@
> -d /var/run/sepermit 0755 root root - -
> +d /run/sepermit 0755 root root - -
> --
> 2.35.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#163371):
> https://lists.openembedded.org/g/openembedded-core/message/163371
> Mute This Topic: https://lists.openembedded.org/mt/89838112/1686489
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> alex.kanavin@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
Ricardo Salveti March 17, 2022, 1:43 p.m. UTC | #2
On Thu, Mar 17, 2022 at 3:53 AM Alexander Kanavin
<alex.kanavin@gmail.com> wrote:
>
> A link to back up the claim would be helpful. Also, what makes this file and setting systemd specific?

This file is installed under tmpfiles.d, which is only used when
systemd is enabled, which is why it is specific to it.

When using /var/run with systemd you will see the following message,
which I can add to the commit message:

systemd-tmpfiles[340]: /etc/tmpfiles.d/pam.conf:1: Line references
path below legacy directory /var/run/, updating /var/run/sepermit →
/run/sepermit; please update the tmpfiles.d/ drop-in file accordingly.

Thanks,
Alexander Kanavin March 17, 2022, 1:48 p.m. UTC | #3
On Thu, 17 Mar 2022 at 14:44, Ricardo Salveti <ricardo@foundries.io> wrote:

> When using /var/run with systemd you will see the following message,
> which I can add to the commit message:
>
> systemd-tmpfiles[340]: /etc/tmpfiles.d/pam.conf:1: Line references
> path below legacy directory /var/run/, updating /var/run/sepermit →
> /run/sepermit; please update the tmpfiles.d/ drop-in file accordingly.

Yes please.

Thanks,
Alex

Patch

diff --git a/meta/recipes-extended/pam/libpam/pam-volatiles.conf b/meta/recipes-extended/pam/libpam/pam-volatiles.conf
index d8b8259a93..1263feb03d 100644
--- a/meta/recipes-extended/pam/libpam/pam-volatiles.conf
+++ b/meta/recipes-extended/pam/libpam/pam-volatiles.conf
@@ -1 +1 @@ 
-d /var/run/sepermit 0755 root root - -
+d /run/sepermit 0755 root root - -