[systemd-devel] [PATCH 1/2] bus-util: add ssh and polkit connection methods

Umut Tezduyar umut at tezduyar.com
Wed Oct 30 21:10:15 CET 2013


Hi,

Speaking about ssh, are sd_bus_open_system_remote() and
bus_connect_system_ssh() doing similar things? bus_connect_system_ssh
expects the "unixexec:" support from dbus library. Can
bus_connect_system_ssh be replaced with sd_bus_open_system_remote so
we won't be dependent on latest (Seems like support has been in since
2012 :) https://bugs.freedesktop.org/show_bug.cgi?id=35230 ) dbus
library.

Thanks

On Wed, Oct 30, 2013 at 3:50 PM, Lennart Poettering
<lennart at poettering.net> wrote:
> On Wed, 30.10.13 06:55, Simon Peeters (peeters.simon at gmail.com) wrote:
>
>> bus_connect_system_ssh is shamelessly copied from Tom Gundersen's wip
>> patches
>
> Tom's patches are merged them and I have now moved the SSH bits into the
> public API of libsystem-bus.
>
> The PolKit transport we decided to remove, as nobody really was using
> it, and it is better to add polkit auth to specific calls rather than
> for the entire transport. Thus, during the conversion all polkit
> transport should be removed from the tools and man pages. (To clarify:
> only the *transport* i.e. the -P switch to the xyzctl tools should
> go,not the general polkit supportfor authenticated operations).
>
> However, I just added support for connecting to the system bus of
> arbitrary local containers. Also in the public API. All of the xyzctl
> tools we have should make use of this. Ultimately this means that the
> polkit transport is just "replaced" by the container transport.
>
> The machinectl tool does this correctly now, it's the example to base
> the other tools on.
>
> Can I convince you to rebase the patches with this in mind?
>
> Lennart
>
> --
> Lennart Poettering, Red Hat
> _______________________________________________
> systemd-devel mailing list
> systemd-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/systemd-devel


More information about the systemd-devel mailing list