Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2016-06-18 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126170/ --- (Updated June 18, 2016, 4:06 a.m.) Status -- This change has been

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2016-06-18 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126170/ --- (Updated June 18, 2016, 10:52 a.m.) Review request for KDE Software on

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2016-01-02 Thread David Faure
> On Dec. 2, 2015, 7:51 a.m., David Faure wrote: > > Please kind in mind that kded must be able to pop up dialogs, though. > > (cookie dialog, SSL cert messagebox + dialog, etc. etc.). > > > > If making it an "agent" doesn't prevent it from showing GUI elements now > > and then, then no

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2016-01-02 Thread David Faure
> On Dec. 2, 2015, 7:51 a.m., David Faure wrote: > > Please kind in mind that kded must be able to pop up dialogs, though. > > (cookie dialog, SSL cert messagebox + dialog, etc. etc.). > > > > If making it an "agent" doesn't prevent it from showing GUI elements now > > and then, then no

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2016-01-02 Thread René J . V . Bertin
> On Dec. 2, 2015, 8:51 a.m., David Faure wrote: > > Please kind in mind that kded must be able to pop up dialogs, though. > > (cookie dialog, SSL cert messagebox + dialog, etc. etc.). > > > > If making it an "agent" doesn't prevent it from showing GUI elements now > > and then, then no

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2016-01-02 Thread René J . V . Bertin
> On Dec. 2, 2015, 8:51 a.m., David Faure wrote: > > Please kind in mind that kded must be able to pop up dialogs, though. > > (cookie dialog, SSL cert messagebox + dialog, etc. etc.). > > > > If making it an "agent" doesn't prevent it from showing GUI elements now > > and then, then no

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-12-25 Thread René J . V . Bertin
> On Dec. 2, 2015, 8:51 a.m., David Faure wrote: > > Please kind in mind that kded must be able to pop up dialogs, though. > > (cookie dialog, SSL cert messagebox + dialog, etc. etc.). > > > > If making it an "agent" doesn't prevent it from showing GUI elements now > > and then, then no

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-12-25 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126170/ --- (Updated Dec. 25, 2015, 10:14 p.m.) Review request for KDE Software on

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-12-12 Thread David Faure
On Monday 07 December 2015 16:14:54 René J.V. Bertin wrote: > On Sunday December 06 2015 14:51:40 David Faure wrote: > > > Here is an easy way to test this: do the same change for kiod in kio (it's > > like a mini kded) and then > > cd kio/tests ; ./listjobtest ftp://t...@upload.kde.org > >

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-12-07 Thread René J . V . Bertin
On Sunday December 06 2015 14:51:40 David Faure wrote: > Here is an easy way to test this: do the same change for kiod in kio (it's > like a mini kded) and then > cd kio/tests ; ./listjobtest ftp://t...@upload.kde.org > should bring up a password dialog. Regardless of what I try (even with

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-12-06 Thread David Faure
> On Dec. 2, 2015, 7:51 a.m., David Faure wrote: > > Please kind in mind that kded must be able to pop up dialogs, though. > > (cookie dialog, SSL cert messagebox + dialog, etc. etc.). > > > > If making it an "agent" doesn't prevent it from showing GUI elements now > > and then, then no

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-12-06 Thread René J . V . Bertin
On Sunday December 06 2015 14:51:40 David Faure wrote: > Here is an easy way to test this: do the same change for kiod in kio (it's > like a mini kded) and then > cd kio/tests ; ./listjobtest ftp://t...@upload.kde.org > should bring up a password dialog. OK, hope to get around to doing that

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-12-06 Thread David Faure
On Sunday 06 December 2015 21:55:44 René J.V. Bertin wrote: > On Sunday December 06 2015 14:51:40 David Faure wrote: > > > Here is an easy way to test this: do the same change for kiod in kio (it's > > like a mini kded) and then > > cd kio/tests ; ./listjobtest ftp://t...@upload.kde.org > >

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-12-02 Thread René J . V . Bertin
> On Dec. 2, 2015, 8:51 a.m., David Faure wrote: > > Please kind in mind that kded must be able to pop up dialogs, though. > > (cookie dialog, SSL cert messagebox + dialog, etc. etc.). > > > > If making it an "agent" doesn't prevent it from showing GUI elements now > > and then, then no

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-12-01 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126170/#review89022 --- Please kind in mind that kded must be able to pop up dialogs,

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-11-29 Thread René J . V . Bertin
On Nov. 25, 2015, 6:18 p.m., René J.V. Bertin wrote: > > See qtbase/src/plugins/platforms/cocoa/qcocoaintegration.mm and > > qcocoahelpers.mm > > René J.V. Bertin wrote: > Isn't copy/paste a great tool? :) > > Anyway, `qt_mac_transformProccessToForegroundApplication` only reads

Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-11-25 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126170/ --- Review request for KDE Software on Mac OS X and KDE Frameworks.

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-11-25 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126170/#review88837 --- Would it be an idea to create a framework function to run the

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-11-25 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126170/#review88838 --- src/kded.cpp (line 683)

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-11-25 Thread René J . V . Bertin
On Nov. 25, 2015, 6:18 p.m., René J.V. Bertin wrote: > > See qtbase/src/plugins/platforms/cocoa/qcocoaintegration.mm and > > qcocoahelpers.mm Isn't copy/paste a great tool? :) Anyway, `qt_mac_transformProccessToForegroundApplication` only reads the `LSUIElement` key to determine whether an

Re: Review Request 126170: [OS X] make kded5 an agent, and build it as a regular application instead of an app bundle

2015-11-25 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126170/ --- (Updated Nov. 25, 2015, 7:12 p.m.) Review request for KDE Software on