> On Jan. 23, 2015, 7:40 a.m., Martin Gräßlin wrote: > > This looks still wrong to me. The service is still called "org.kde.kwin" > > while it should be "org.kde.KWin". Ideally this gets changed to a generated > > adaptor from the DBus interface KWin installs. > > > > To make things worse: the service name is not guaranteed to be > > "org.kde.KWin" - it can be mangled on multi-head systems. I had considered > > eporting it to a root window property, maybe I should implement that. > > Martin Gräßlin wrote: > And just implemented the announcment of the DBus service: > https://git.reviewboard.kde.org/r/122215/ > > Ivan Čukić wrote: > How will this work? > > KAMD is a unique instance per dbus session, kwin can have more instances. > Which instance handles the session stuff? etc. > > I don't like the idea of introducing another connection from kamd to X. > > Martin Gräßlin wrote: > If KAMD doesn't want to support multi-head (which is fine) it should pick > the kwin instance responsible for the X head it's on.
Sorry, I need more details here. What confuses me is the 'X head it's on' - it is a service, it has no UI to be on a specific head. Or am I misunderstanding what a 'head' is? :) Can you describe (nothing detailed - just a few words) the situation when kwin has multiple instances / multiple dbus services. Does it imply separate X sessions? - Ivan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122065/#review74574 ----------------------------------------------------------- On Jan. 22, 2015, 11:42 p.m., Xuetian Weng wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122065/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2015, 11:42 p.m.) > > > Review request for Plasma, Martin Gräßlin and Ivan Čukić. > > > Repository: kactivities > > > Description > ------- > > 1. KWin Interface name is wrong > 2. kactivitymanagerd doesn't listen on ksmserver anymore, thus > subSession{Opened,Closed,CloseCanceled} are not called anymore. It need to > send Started/Stopped event by itself when kwin dbus call returns. > > > Diffs > ----- > > src/service/ksmserver/KSMServer_p.h c0f5598 > src/service/ksmserver/KSMServer.cpp b5e1467 > > Diff: https://git.reviewboard.kde.org/r/122065/diff/ > > > Testing > ------- > > Now starting and stopping activity works properly. > > > Thanks, > > Xuetian Weng > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
