[KDE/Mac] Review Request 120510: [OS X] make debugger build functional

[René J.V. Bertin]

> On Oct. 16, 2014, 11:47 a.m., Milian Wolff wrote:
> > debuggers/gdb/stty.cpp, line 156
> > <https://git.reviewboard.kde.org/r/120510/diff/3/?file=319960#file319960line156>
> >
> >     now this can be left at -1, no?

Indeed (I really ought to get more sleep ...)


> On Oct. 16, 2014, 11:47 a.m., Milian Wolff wrote:
> > debuggers/gdb/gdb.cpp, line 258
> > <https://git.reviewboard.kde.org/r/120510/diff/3/?file=319959#file319959line258>
> >
> >     join line with next

Ok, but I'm a bit surprised; I tried to follow the existing style as closely as possible (I never code like this except when the conditional statement is part of some #ifdef, to avoid duplicating the opening brace). Should I bring the other places "up to style" too?


- René J.V.


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120510/#review68542
-----------------------------------------------------------


On Oct. 15, 2014, 10:52 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120510/
> -----------------------------------------------------------
> 
> (Updated Oct. 15, 2014, 10:52 p.m.)
> 
> 
> Review request for KDE Software on Mac OS X and KDevelop.
> 
> 
> Repository: kdevelop
> 
> 
> Description
> -------
> 
> The conditions are united under OS X / Macports to build kdevelop's debugger component: the changes to the CMake files take this into account.
> 
> I have made the required changes in stty.cpp to obtain a pty with r/w permissions - on OS X this is done in a way that shares most code with the SGI code. In order not to duplicate more code than necessary I changed the logic of `ptyfd`'s initial value slightly, allowing to distinguish between uninitialised and error return values.
> 
> One has to install an uptodate gdb version (e.g. through MacPorts) and follow the instructions to give it the required permissions to function. Once that's done, the debugger component starts correctly, and appears to communicate with the gdb ("ggdb") slave process.
> 
> The gdb version in MacPorts does not react to SIGKILL, which is the signal used by KDevelop to interrupt a programme being debugged. I solved that issue by retrieving the debugged application's pid from the gdb output, and sending the signal to that application instead of to gdb.
> 
> 
> Diffs
> -----
> 
>   debuggers/CMakeLists.txt 8fe222c 
>   debuggers/gdb/CMakeLists.txt 3d1125c 
>   debuggers/gdb/gdb.h 6f99a60 
>   debuggers/gdb/gdb.cpp 68e1768 
>   debuggers/gdb/stty.cpp 736fff4 
> 
> Diff: https://git.reviewboard.kde.org/r/120510/diff/
> 
> 
> Testing
> -------
> 
> OS X 10.6.8 with kdelibs 4.14.1 and kdesdk git/kde4-legacy . Gdb 7.7.1 from MacPorts, compiled with python 2.7 support.
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-mac/attachments/20141016/1b539442/attachment.html>