[PATCH weston] build: install a login manager session entry

sardemff7+wayland at sardemff7.net sardemff7+wayland at sardemff7.net
Wed Jan 21 03:53:23 PST 2015


On 2015-01-21 04:46, Bryce Harrington wrote:
> On Sun, Dec 28, 2014 at 02:20:49PM +0100, Lubomir Rintel wrote:
>> This makes it possible to run Weston session from GDM (and possibly
>> other login managers).
>> ---
>>   Makefile.am        | 5 +++++
>>   src/weston.desktop | 5 +++++
>>   2 files changed, 10 insertions(+)
>>   create mode 100644 src/weston.desktop
>
> Thanks, applied
>     1a873aa..4f4b47a  master -> master


Seems like I failed to reply to the list for my review on this one.

Basically, you should (read “must”) use dist_wayland_session_DATA here
and not EXTRA_DIST (because the dist_* variant works perfectly).
Should I send a follow-up or would you push the trivial fix yourself?


Also I was wondering about the whole weston to run in the D-Bus session
since it may be more sensible to run the weston-desktop-shell only.
Another “Weston DE” consideration here I guess.

Cheers,

-- 

Quentin “Sardem FF7” Glidic


More information about the wayland-devel mailing list