Re: Review Request 122065: Fix activity start and stop.

2015-02-01 Thread Ivan Čukić
On Jan. 27, 2015, 4:25 p.m., Ivan Čukić wrote: Ship It! @Xuetian Weng Do you have commit access, or should I push this? - Ivan --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 122065: Fix activity start and stop.

2015-02-01 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122065/ --- (Updated Feb. 1, 2015, 10:39 p.m.) Status -- This change has been

Re: Review Request 122065: Fix activity start and stop.

2015-01-27 Thread Ivan Čukić
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122065/#review74835 --- Ship it! Ship It! - Ivan Čukić On Jan. 27, 2015, 4:18

Re: Review Request 122065: Fix activity start and stop.

2015-01-27 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122065/ --- (Updated Jan. 27, 2015, 4:18 p.m.) Review request for Plasma, Martin

Re: Review Request 122065: Fix activity start and stop.

2015-01-26 Thread Ivan Čukić
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

Re: Review Request 122065: Fix activity start and stop.

2015-01-26 Thread Martin Gräßlin
On Jan. 23, 2015, 8: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

Re: Review Request 122065: Fix activity start and stop.

2015-01-26 Thread Ivan Čukić
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

Re: Review Request 122065: Fix activity start and stop.

2015-01-26 Thread Ivan Čukić
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122065/#review74808 --- src/service/ksmserver/KSMServer.cpp

Re: Review Request 122065: Fix activity start and stop.

2015-01-26 Thread Martin Gräßlin
On Jan. 27, 2015, 8:27 a.m., Ivan Čukić wrote: src/service/ksmserver/KSMServer.cpp, line 72 https://git.reviewboard.kde.org/r/122065/diff/2/?file=345332#file345332line72 Can you add: - checking for the environment variable existence (and using it, if not empty) - a

Re: Review Request 122065: Fix activity start and stop.

2015-01-26 Thread Ivan Čukić
On Jan. 27, 2015, 7:27 a.m., Ivan Čukić wrote: src/service/ksmserver/KSMServer.cpp, line 72 https://git.reviewboard.kde.org/r/122065/diff/2/?file=345332#file345332line72 Can you add: - checking for the environment variable existence (and using it, if not empty) - a

Re: Review Request 122065: Fix activity start and stop.

2015-01-26 Thread Martin Gräßlin
On Jan. 27, 2015, 8:27 a.m., Ivan Čukić wrote: src/service/ksmserver/KSMServer.cpp, line 72 https://git.reviewboard.kde.org/r/122065/diff/2/?file=345332#file345332line72 Can you add: - checking for the environment variable existence (and using it, if not empty) - a

Re: Review Request 122065: Fix activity start and stop.

2015-01-26 Thread Xuetian Weng
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

Re: Review Request 122065: Fix activity start and stop.

2015-01-26 Thread Martin Gräßlin
On Jan. 23, 2015, 8: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

Re: Review Request 122065: Fix activity start and stop.

2015-01-26 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122065/ --- (Updated Jan. 27, 2015, 12:23 a.m.) Review request for Plasma, Martin

Re: Review Request 122065: Fix activity start and stop.

2015-01-25 Thread Martin Gräßlin
On Jan. 23, 2015, 8: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

Re: Review Request 122065: Fix activity start and stop.

2015-01-24 Thread Martin Gräßlin
On Jan. 23, 2015, 8: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

Re: Review Request 122065: Fix activity start and stop.

2015-01-23 Thread Xuetian Weng
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

Re: Review Request 122065: Fix activity start and stop.

2015-01-23 Thread Xuetian Weng
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

Re: Review Request 122065: Fix activity start and stop.

2015-01-23 Thread Martin Gräßlin
On Jan. 23, 2015, 8: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

Re: Review Request 122065: Fix activity start and stop.

2015-01-23 Thread Martin Gräßlin
On Jan. 23, 2015, 8: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

Re: Review Request 122065: Fix activity start and stop.

2015-01-23 Thread Ivan Čukić
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

Re: Review Request 122065: Fix activity start and stop.

2015-01-23 Thread Ivan Čukić
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

Re: Review Request 122065: Fix activity start and stop.

2015-01-23 Thread Martin Gräßlin
On Jan. 23, 2015, 8: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

Re: Review Request 122065: Fix activity start and stop.

2015-01-22 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122065/#review74574 --- This looks still wrong to me. The service is still called

Review Request 122065: Fix activity start and stop.

2015-01-14 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122065/ --- Review request for Plasma and Ivan Čukić. Repository: kactivities