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

Move D-Bus conf file to share/dbus-1/system.d #492

Closed
wants to merge 1 commit into from

Conversation

worldofpeace
Copy link

Since D-Bus 1.9.18 configuration files installed by third-party should
go in share/dbus-1/system.d. The old location is for sysadmin overrides.

@aschnell
Copy link
Member

aschnell commented Sep 16, 2019

Thanks for the contribution.

Since when does D-Bus also check in /usr/share? snapper is also provided for older distributions (see https://build.opensuse.org/package/show/filesystems:snapper/snapper) and those must not break.

@worldofpeace
Copy link
Author

Since 1.9.18 released in 2015. Which distro's do you deliver to that have DBus older than this?

@aschnell
Copy link
Member

In the build project I mentioned there are some, e.g. SLE12 SP4, RHEL 7, Fedora 22, Debian 8 or Ubuntu 15.04 all use dbus <= 1.8. At least SLE12 SP4 is still maintained by SUSE.

Is there no way to ask dbus about configuration settings at build time?

@worldofpeace
Copy link
Author

I will see if I can arrange something in configure.ac to use the old directory when it doesn't match that version.

@worldofpeace
Copy link
Author

How does this seem?

AC_ARG_WITH(dbus-sys, [  --with-dbus-sys=<dir>   where D-BUS system.d directory is])
if ! test -z "$with_dbus_sys" ; then
    DBUS_SYS_DIR="$with_dbus_sys"
else
    DBUS_SYS_DIR="$datadir/dbus-1/system.d"
fi
AC_SUBST(DBUS_SYS_DIR)

And in your specs you can use --with-dbus-sys=/etc/dbus-1/system.d for older targets?

@aschnell
Copy link
Member

Yes, as a last resort setting the path like that should work.

@mvidner
Copy link
Member

mvidner commented Oct 14, 2019

Hi @worldofpeace , I don't know about building on Debian but I managed to fix the build failure anyway :) 67d299b

@worldofpeace
Copy link
Author

Hi @worldofpeace , I don't know about building on Debian but I managed to fix the build failure anyway :) 67d299b

Thanks, will integrate 👍

Since D-Bus 1.9.18 configuration files installed by third-party should
go in share/dbus-1/system.d. The old location is for sysadmin overrides.

Co-authored-by: Martin Vidner <mvidner@suse.cz>
@aschnell
Copy link
Member

aschnell commented Apr 8, 2021

Meanwhile the distributions where dbus is too old are dropped for other reasons for snapper. So the conditional stuff is not needed more.

But at least on SLE 15 the directory /usr/share/dbus-1/system.d is not yet owned by systemd. I'm cheking all supported distributions right now.

@aschnell
Copy link
Member

aschnell commented Apr 8, 2021

Implemented with #634.

@aschnell aschnell closed this Apr 8, 2021
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.

None yet

3 participants