Re: Review Request 119497: Report crashes of KDE apps in Apple OS X (1) (fix kcrash)

2014-09-15 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119497/#review66509 --- Ship it! The wrong socket name sounds like a workaround for

Re: Review Request 119497: Report crashes of KDE apps in Apple OS X (1) (fix kcrash)

2014-09-15 Thread René J . V . Bertin
On July 27, 2014, 1:32 p.m., Thomas Lübking wrote: kdeui/util/kcrash.cpp, line 316 https://git.reviewboard.kde.org/r/119497/diff/1/?file=293441#file293441line316 is libdispatch OSX only or is it also used on FreeBSD? The question (more to Michael ;-) is whether

Re: Review Request 120149: [OS X] improved menubar experience: protected Preferences menu and cleaner system tray

2014-09-15 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120149/ --- (Updated Sept. 15, 2014, 1:03 p.m.) Review request for KDE Software on

Re: Review Request 120149: [OS X] improved menubar experience: protected Preferences menu and cleaner system tray

2014-09-15 Thread René J . V . Bertin
On Sept. 15, 2014, 2:45 a.m., Ian Wadham wrote: I cannot see how static void setTextWithCorrectMenuRole() could work in all languages. Does not the text parameter come translated into the user's language, not necessarily English? So how can the .contains checks work with all

Re: Review Request 120149: [OS X] improved menubar experience: protected Preferences menu and cleaner system tray

2014-09-15 Thread Thomas Lübking
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120149/#review66546 --- In general coding style is systematically wrong ( if (a, not

Re: Review Request 120149: [OS X] improved menubar experience: protected Preferences menu and cleaner system tray

2014-09-15 Thread René J . V . Bertin
On Sept. 15, 2014, 2:19 p.m., Thomas Lübking wrote: kdeui/actions/kaction.cpp, line 188 https://git.reviewboard.kde.org/r/120149/diff/2/?file=312029#file312029line188 given KAction, KMenu and KMenuBar are all deprecated in KF5, is this actually of any upstream relevance? I

Re: Review Request 120149: [OS X] improved menubar experience: protected Preferences menu and cleaner system tray

2014-09-15 Thread René J . V . Bertin
On Sept. 15, 2014, 2:19 p.m., Thomas Lübking wrote: kdeui/actions/kaction.cpp, line 164 https://git.reviewboard.kde.org/r/120149/diff/2/?file=312029#file312029line164 would this eg. work with kwrite (Configure Editor...)? - or other kpart driven things? René J.V. Bertin wrote:

Re: Using Gerrit for code review in KDE

2014-09-15 Thread Jan Kundrát
On Saturday, 13 September 2014 23:05:48 CEST, Eike Hein wrote: Yeah, that's something I'm OK with too. Maybe we can even adapt the UI to use strings like Sven proposes? https://gerrit.vesnicky.cesnet.cz/r/35 With kind regards, Jan -- Trojitá, a fast Qt IMAP e-mail client --

Re: Using Gerrit for code review in KDE

2014-09-15 Thread Milian Wolff
On Saturday 13 September 2014 23:05:48 Eike Hein wrote: On 13.09.2014 22:50, Sven Brauch wrote: That's my opinion as well. It would be nice to have an explicit way to differentiate the I think this patch is okay, but I'm not very familiar with the code you changed (+1) and I'm confident

Re: Using Gerrit for code review in KDE

2014-09-15 Thread Thiago Macieira
On Monday 15 September 2014 16:49:39 Milian Wolff wrote: Where do I see the diff there? In the gerrit that runs on qt-project, I can easily click one button to go to a unified or side-by-side diff view. Is that a custom extension? Generally, it seems as if the qt-project gerrit has a much

Re: Review Request 120149: [OS X] improved menubar experience: protected Preferences menu and cleaner system tray

2014-09-15 Thread René J . V . Bertin
On Sept. 15, 2014, 2:19 p.m., Thomas Lübking wrote: kdeui/widgets/kmenu.cpp, line 174 https://git.reviewboard.kde.org/r/120149/diff/2/?file=312033#file312033line174 this is *utterly* wrong - you're manipulating a QAction reference just because it (at this very time!) hints it

Re: Review Request 119497: Report crashes of KDE apps in Apple OS X (1) (fix kcrash)

2014-09-15 Thread Marko Käning
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119497/#review66598 --- Hi Ian, shall I test this on a Mavericks VM before you're

Re: Review Request 120149: [OS X] improved menubar experience: protected Preferences menu and cleaner system tray

2014-09-15 Thread Thomas Lübking
On Sept. 15, 2014, 12:19 nachm., Thomas Lübking wrote: kdeui/actions/kaction.cpp, line 149 https://git.reviewboard.kde.org/r/120149/diff/2/?file=312029#file312029line149 what if KAction *foo = new KAction(this); foo-setText(Foo); - you rather want to monitor

Re: Using Gerrit for code review in KDE

2014-09-15 Thread Jan Kundrát
On Monday, 15 September 2014 16:49:39 CEST, Milian Wolff wrote: Where do I see the diff there? For me, it's easiest to just click on any file name. That will open a diff view (either side-by-side or a unidiff one, based on your prefs). The diff shows just a single file, but you can use [ and

Automatic Info Reporting BoF summary

2014-09-15 Thread Albert Astals Cid
BoF that that is a follow up of https://conf.kde.org/system/attachments/45/original/spyware.pdf?1410020392 and http://files.kde.org/akademy/2014/videos/The_case_for_spyware_-_Albert_Astals_Cid.webm Albert's plan. 1. Have a KDED. --- On First Run - Show Dialog --- Send data every

Re: Review Request 120149: [OS X] improved menubar experience: protected Preferences menu and cleaner system tray

2014-09-15 Thread René J . V . Bertin
On Sept. 15, 2014, 2:19 p.m., Thomas Lübking wrote: kdeui/actions/kaction.cpp, line 164 https://git.reviewboard.kde.org/r/120149/diff/2/?file=312029#file312029line164 would this eg. work with kwrite (Configure Editor...)? - or other kpart driven things? René J.V. Bertin wrote:

Re: Review Request 120210: kdoctools: style images more elegantly

2014-09-15 Thread T.C. Hollingsworth
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120210/ --- (Updated Sept. 16, 2014, 1:15 a.m.) Review request for Documentation,

Re: Review Request 119497: Report crashes of KDE apps in Apple OS X (1) (fix kcrash)

2014-09-15 Thread Ian Wadham
On Sept. 15, 2014, 7:26 p.m., Marko Käning wrote: Hi Ian, shall I test this on a Mavericks VM before you're committing this? In case I shall do it, do you have a test case for me? Greets, Marko Thanks, but please wait until I drop the other shoe, review 119498, fix Dr Konqi. The test

Re: Review Request 119497: Report crashes of KDE apps in Apple OS X (1) (fix kcrash)

2014-09-15 Thread Marko Käning
On Sept. 15, 2014, 9:26 p.m., Marko Käning wrote: Hi Ian, shall I test this on a Mavericks VM before you're committing this? In case I shall do it, do you have a test case for me? Greets, Marko Ian Wadham wrote: Thanks, but please wait until I drop the other shoe, review 119498,

Re: Review Request 119497: Report crashes of KDE apps in Apple OS X (1) (fix kcrash)

2014-09-15 Thread Marko Käning
On Sept. 15, 2014, 9:26 p.m., Marko Käning wrote: Hi Ian, shall I test this on a Mavericks VM before you're committing this? In case I shall do it, do you have a test case for me? Greets, Marko Ian Wadham wrote: Thanks, but please wait until I drop the other shoe, review 119498,

Re: Review Request 119498: Report crashes of KDE apps in Apple OS X (2) (fix drkonqi)

2014-09-15 Thread Marko Käning
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119498/#review66626 --- And this should be marked as depending on RR 119497 in