Re: Review Request 125648: Add method to run a script from DBus directly

2016-05-03 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/ --- (Updated May 3, 2016, 11:01 a.m.) Status -- This change has been mar

Re: Review Request 125648: Add method to run a script from DBus directly

2016-04-29 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/#review95005 --- Ship it! last change i would like is to not run the script

Re: Review Request 125648: Add method to run a script from DBus directly

2016-04-28 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/#review94983 --- bump Marco you seemed to be leaning towards this, can I have

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-19 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/#review87077 --- KWin supports loading scripts at runtime through a dbus interf

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-19 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/#review87074 --- FWIW, I know this was deliberately not implemented in the past

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-19 Thread Marco Martin
> On Oct. 19, 2015, 10:43 a.m., Marco Martin wrote: > > is true that for a shell script would already be possible to poke with the > > config file.. > > but wouldn't this be a bit too much of a security hole/open invitation for > > anybody, script or app to poke in the desktop configuration wit

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-19 Thread David Edmundson
> On Oct. 19, 2015, 10:43 a.m., Marco Martin wrote: > > is true that for a shell script would already be possible to poke with the > > config file.. > > but wouldn't this be a bit too much of a security hole/open invitation for > > anybody, script or app to poke in the desktop configuration wit

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-19 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/#review87061 --- is true that for a shell script would already be possible to p

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-15 Thread David Edmundson
> On Oct. 15, 2015, 5:26 p.m., Martin Klapetek wrote: > > shell/shellcorona.cpp, line 1134 > > > > > > Should there be at least some basic input treatment to prevent > > malicious activity? > > > > Run

Re: Review Request 125648: Add method to run a script from DBus directly

2015-10-15 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125648/#review86888 --- shell/shellcorona.cpp (line 1134)