[PATCH wayland 0/6] Add wl_object based custom dispatchers support

John Kåre Alsaker john.kare.alsaker at gmail.com
Sun Mar 17 03:28:37 PDT 2013


I suggest we apply all of them except the last 4 which add dispatcher
support. That allows us to slowly fix the other projects with the new
API.

On Sat, Mar 16, 2013 at 9:25 PM, Scott Moreau <oreaus at gmail.com> wrote:
> Hi Jason,
>
> I appreciate your work here. These patches can potentially lower the
> entry barrier to writing other language bindings. Unfortunately, I
> don't not feel that this is something suitable for gh next. It adds a
> dependency on a mesa patch, which makes it more difficult to build the
> next stack, without many immediate benefits. I wanted to make it clear
> upfront to avoid wasted effort. I am glad that we were able to get
> many of the bugs worked out before it is reviewed, it is nice to see
> the community working together here.
>
> - Scott
>
> On Sat, Mar 16, 2013 at 1:15 PM, Jason Ekstrand <jason at jlekstrand.net> wrote:
>> Kristian,
>> Working with Scott Moreau to get this applied to his "next" branch
>> demonstrated that this series is a bit tricky to apply.  In order to
>> do everything correctly, it needs to be applied in three stages:
>>
>> First, apply the following 4 patches (in order):
>> http://lists.freedesktop.org/archives/wayland-devel/2013-March/007830.html
>> http://lists.freedesktop.org/archives/wayland-devel/2013-March/007964.html
>> http://lists.freedesktop.org/archives/wayland-devel/2013-March/007844.html
>> http://lists.freedesktop.org/archives/wayland-devel/2013-March/007846.html
>>
>> The last of the above patches adds a wl_resource_init function for
>> initializing the wl_resource structure.  After this, Weston, Mesa, and
>> (possibly) xwayland need to be patched to use wl_resource_init.  The
>> following patch for weston applies against git master, but will
>> probably need to be re-done due to changes made in weston:
>> http://lists.freedesktop.org/archives/wayland-devel/2013-March/007845.html
>>
>> Finally, once Weston, xwayland, and Mesa have been patched to use
>> wl_resource_init, we can apply the last 4 patches in the series (these
>> actually add dispatcher support):
>>
>> http://lists.freedesktop.org/archives/wayland-devel/2013-March/007850.html
>> http://lists.freedesktop.org/archives/wayland-devel/2013-March/007848.html
>> http://lists.freedesktop.org/archives/wayland-devel/2013-March/007849.html
>> http://lists.freedesktop.org/archives/wayland-devel/2013-March/007847.html
>>
>> Hopefully, this will keep the transition from being too rocky.
>> --Jason Ekstrand
>> _______________________________________________
>> wayland-devel mailing list
>> wayland-devel at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list