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

Hans de Goede hdegoede at redhat.com
Sat Jan 18 08:27:58 PST 2014


Hi,

On 01/17/2014 02:08 PM, Daniel Stone wrote:
> Hi,
>
> On 15 January 2014 14:32, Hans de Goede <hdegoede at redhat.com> wrote:
>> With systemd-logind the dbus-core will be used for more then just config.
>>
>> This patch also makes the building of the dbus-core and udev no longer
>> mutually exclusive and removes the config_ prefix from the dbus-core symbols.
>
> The rest of your patchset looks good to me, but I'd rather this not
> get moved to hw/xfree86/.  Historically, hw/xfree86/ has been a bit of
> a ghetto that we've tried to move things out of, because the majority
> of code in there is really terrible.  It also means you can't use HAL
> support on non-XFree86 DDXes, which I admit is entirely a theoretical
> concern atm, but still ...

Suggestions for where it should be moved instead then ? Putting it in the os
dir means all kinda binaries which are not using dbus at all start depending
on it too... Maybe leave it in config (while keeping the other changes ?

Regards,

Hans


More information about the xorg-devel mailing list