[PATCH 01/12] dbus-core: Move to hw/xfree86/common dir

Daniel Stone daniel at fooishbar.org
Thu Jan 23 04:56:36 PST 2014


Hi,

On 23 January 2014 11:16, Hans de Goede <hdegoede at redhat.com> wrote:
> Ok, I've put it back in config (while keeping the other changes like making
> it no longer mutually exclusive with udev). I've pushed the updated version
> here:
> http://cgit.freedesktop.org/~jwrdegoede/xserver

Thanks for this.

bb992fbd config: drop the dbus API
    This should've been removed a long time ago.

c68423c0 dbus-core: Attempt to connect to dbus ASAP
    Seems fine to me, assuming that all users of the hook don't assume
we're further along in the init process than we are.

738b1e91 dbus-core: Make dbus-core no longer mutually exclusive with udev
    Sorry to keep bikeshedding, but I do worry slightly about taking
over the dbus_core_*() namespace.  Should be safe in theory, since I
doubt anyone's going to be adding to the D-Bus C API anytime soon, but
...
    Also, we could move dbus_core_init() out of xf86Init, into
dix/main.c's loop around the same time.

8d972e0c xf86Xinput: Modify API for server-managed fd support
    I worry a little that drivers won't necessarily support this -
would be nice to check by forcing them to define
I_SUPPORT_SERVER_MANAGED_FDS or something; either that, or extend
InputDriverRec with an equivalent flag which must be set.

With those fixed, those four look good to me, and:
Reviewed-by: Daniel Stone <daniel at fooishbar.org>

Cheers,
Daniel


More information about the xorg-devel mailing list