Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

base: dbus: specify runstatedir to use /run #1000

Merged
merged 1 commit into from
Jan 9, 2023

Conversation

ricardosalveti
Copy link
Member

From oe-core master (4df1a16e5c38d0fb724f63d37cc032aa37fa122f), specify runstatedir to use /run directly (instead of the default /var/run), to fix the following runtime warning from systemd:

systemd-tmpfiles[350]: /usr/lib/tmpfiles.d/dbus.conf:13: Line references path below legacy directory /var/run/, updating /var/run/dbus/containers → /run/dbus/containers; please update the tmpfiles.d/ drop-in file accordingly.

Signed-off-by: Ricardo Salveti ricardo@foundries.io

From oe-core master (4df1a16e5c38d0fb724f63d37cc032aa37fa122f), specify
runstatedir to use /run directly (instead of the default /var/run), to
fix the following runtime warning from systemd:

systemd-tmpfiles[350]: /usr/lib/tmpfiles.d/dbus.conf:13: Line references
path below legacy directory /var/run/, updating /var/run/dbus/containers →
/run/dbus/containers; please update the tmpfiles.d/ drop-in file
accordingly.

Signed-off-by: Ricardo Salveti <ricardo@foundries.io>
Copy link
Member

@quaresmajose quaresmajose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, also good to send upstream

Copy link
Contributor

@MrCry0 MrCry0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. +1 for upstreaming

@ricardosalveti
Copy link
Member Author

LGTM, also good to send upstream

This one is already set in master (it wasn't backported on purpose it seems by sakoman).

Copy link
Contributor

@angolini angolini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should start thinking about a way to organize those bits we know are going to be dropped in the next stable.

Other than that. LGTM

@ricardosalveti ricardosalveti merged commit c4bd1b9 into foundriesio:main Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants