Skip to content
This repository was archived by the owner on Sep 24, 2020. It is now read-only.

Conversation

@crawford
Copy link

@crawford crawford commented Dec 3, 2015

This reverts commit 1544291.

The patch this reverts has flawed logic which ended up being fixed by systemd@751090c. Both patches depend on bus_container_connect_kernel() returning specific error codes (e.g. -ENOENT or -ESOCKTNOSUPPORT), but oddly enough, that functionality didn't exist until systemd@80b0d3e (here and here). Rather than backporting all of those changes (and probably breaking more stuff), use the previous behavior and allow sd-bus to fall back from a kernel connection to a unix socket.

Fixes coreos/bugs#1002.

@mischief
Copy link

mischief commented Dec 3, 2015

lgtm

1 similar comment
@vcaputo
Copy link

vcaputo commented Dec 3, 2015

lgtm

@crawford
Copy link
Author

crawford commented Dec 3, 2015

Fixes coreos/bugs#863.

crawford added a commit that referenced this pull request Dec 3, 2015
Revert "sd-bus: do not connect to dbus-1 socket when kdbus is available"
@crawford crawford merged commit ea53582 into coreos:v225-coreos Dec 3, 2015
@crawford crawford deleted the kdbus branch December 3, 2015 23:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants