Re: Review Request 128941: ZLIB dependency is in libkonq since 7635179

2016-09-18 Thread Rolf Eike Beer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128941/#review99266 --- konqueror/src/CMakeLists.txt

Re: Review Request 128909: initial, minimal support for OS X

2016-09-18 Thread René J . V . Bertin
> On Sept. 17, 2016, 9:52 p.m., René J.V. Bertin wrote: > > processui/ReniceDlg.cpp, lines 78-87 > > > > > > I've actually managed to get ksysguardprocesslist_helper to work with a > > patched libdbus and a hack

Review Request 128941: ZLIB dependency is in libkonq since 7635179

2016-09-18 Thread Andreas Sturmlechner
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128941/ --- Review request for KDE Base Apps and David Faure. Repository:

Re: Review Request 128909: initial, minimal support for OS X

2016-09-18 Thread Aleix Pol Gonzalez
> On Sept. 17, 2016, 9:52 p.m., René J.V. Bertin wrote: > > processui/ReniceDlg.cpp, lines 78-87 > > > > > > I've actually managed to get ksysguardprocesslist_helper to work with a > > patched libdbus and a hack

Re: Review Request 128684: Proofread + update khtml-general kcm docbook

2016-09-18 Thread Burkhard Lück
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128684/ --- (Updated Sept. 18, 2016, 6:05 nachm.) Status -- This change has

Re: Review Request 128803: Import konqueror kcm docbooks from kde-runtime master

2016-09-18 Thread Burkhard Lück
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128803/ --- (Updated Sept. 18, 2016, 6:04 nachm.) Status -- This change has

Re: Review Request 128737: Update second part of konqueror docbook to 16.12

2016-09-18 Thread Burkhard Lück
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128737/ --- (Updated Sept. 18, 2016, 8 p.m.) Status -- This change has been

Re: Review Request 128684: Proofread + update khtml-general kcm docbook

2016-09-18 Thread David Faure
> On Aug. 16, 2016, 7:05 a.m., David Faure wrote: > > doc/kcontrol/khtml-general/index.docbook, line 7 > > > > > > Yeah I don't understand what this docbook is doing here :-) > > Burkhard Lück wrote: > these

Re: Review Request 128803: Import konqueror kcm docbooks from kde-runtime master

2016-09-18 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128803/#review99256 --- Yes, kde-baseapps should be split. Can you move all this to

Re: Review Request 128737: Update second part of konqueror docbook to 16.12

2016-09-18 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128737/#review99255 --- Fix it, then Ship it! Please push after the suggested