Re: Review Request 124304: Reduce the KDBusService timeout from 5 minutes to 25 seconds

2015-07-10 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124304/#review82311 --- Guys, shortening this value won't make anything faster.

Re: Review Request 124304: Reduce the KDBusService timeout from 5 minutes to 25 seconds

2015-07-10 Thread Martin Klapetek
On July 10, 2015, 11:03 a.m., David Faure wrote: Guys, shortening this value won't make anything faster. Instead, in case an app takes 26 seconds to start (slow machine, busy system, lots of initialization code...) the caller will get an error message erroneously. Did you never see

Re: Review Request 124304: Reduce the KDBusService timeout from 5 minutes to 25 seconds

2015-07-10 Thread David Faure
On July 10, 2015, 9:03 a.m., David Faure wrote: Guys, shortening this value won't make anything faster. Instead, in case an app takes 26 seconds to start (slow machine, busy system, lots of initialization code...) the caller will get an error message erroneously. Did you never see

Re: Review Request 124304: Reduce the KDBusService timeout from 5 minutes to 25 seconds

2015-07-10 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124304/ --- (Updated July 10, 2015, 11:52 a.m.) Status -- This change has been

Re: Review Request 124304: Reduce the KDBusService timeout from 5 minutes to 25 seconds

2015-07-10 Thread Martin Klapetek
On July 10, 2015, 11:03 a.m., David Faure wrote: Guys, shortening this value won't make anything faster. Instead, in case an app takes 26 seconds to start (slow machine, busy system, lots of initialization code...) the caller will get an error message erroneously. Did you never see

Re: Review Request 124304: Reduce the KDBusService timeout from 5 minutes to 25 seconds

2015-07-09 Thread Milian Wolff
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124304/#review82275 --- also +1 from my side - Milian Wolff On July 9, 2015, 11:48

Re: Review Request 124304: Reduce the KDBusService timeout from 5 minutes to 25 seconds

2015-07-09 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124304/#review82277 --- a clear +1 - not having blocking dbus calls taking 5 minutes

Review Request 124304: Reduce the KDBusService timeout from 5 minutes to 25 seconds

2015-07-09 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124304/ --- Review request for KDE Frameworks and David Faure. Repository:

Re: Review Request 124304: Reduce the KDBusService timeout from 5 minutes to 25 seconds

2015-07-09 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124304/#review82262 --- +1 makes sense to me. - Aleix Pol Gonzalez On July 9,