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

2016-09-15 Thread René J . V . Bertin
> On Sept. 15, 2016, 1:29 a.m., Aleix Pol Gonzalez wrote: > > processcore/helper.cpp, line 133 > > > > > > `KAUTH_HELPER_MAIN` doesn't work on OS X? > > René J.V. Bertin wrote: > Thanks for picking up on thi

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

2016-09-15 Thread René J . V . Bertin
> On Sept. 15, 2016, 1:29 a.m., Aleix Pol Gonzalez wrote: > > processcore/helper.cpp, line 133 > > > > > > `KAUTH_HELPER_MAIN` doesn't work on OS X? > > René J.V. Bertin wrote: > Thanks for picking up on thi

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

2016-09-15 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128909/#review99214 --- +1 processcore/processes_local_p.cpp (line 34)

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

2016-09-15 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128909/ --- (Updated Sept. 15, 2016, 2:47 p.m.) Review request for KDE Base Apps and

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

2016-09-15 Thread René J . V . Bertin
> On Sept. 15, 2016, 1:29 a.m., Aleix Pol Gonzalez wrote: > > processcore/helper.cpp, line 133 > > > > > > `KAUTH_HELPER_MAIN` doesn't work on OS X? > > René J.V. Bertin wrote: > Thanks for picking up on thi

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

2016-09-15 Thread Aleix Pol Gonzalez
> On Sept. 15, 2016, 1:29 a.m., Aleix Pol Gonzalez wrote: > > processcore/helper.cpp, line 133 > > > > > > `KAUTH_HELPER_MAIN` doesn't work on OS X? > > René J.V. Bertin wrote: > Thanks for picking up on thi

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

2016-09-15 Thread René J . V . Bertin
> On Sept. 15, 2016, 1:29 a.m., Aleix Pol Gonzalez wrote: > > processcore/helper.cpp, line 133 > > > > > > `KAUTH_HELPER_MAIN` doesn't work on OS X? > > René J.V. Bertin wrote: > Thanks for picking up on thi

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

2016-09-15 Thread René J . V . Bertin
> On Sept. 15, 2016, 1:29 a.m., Aleix Pol Gonzalez wrote: > > Other than that, the patch LGTM. Aleix, you coined the idea of implementing a simpler widget that provides just a list of PIDs with their command names and possible command lines, based on the `ps` command (and comparable to the one