#291 closed task (fixed)
add a debug build option to setup.py
Reported by: | Antoine Martin | Owned by: | Antoine Martin |
---|---|---|---|
Priority: | major | Milestone: | 0.9 |
Component: | android | Version: | trunk |
Keywords: | Cc: |
Description (last modified by )
So we can get debug symbols from the cython bits.
And add flag handling so we can enable it with --with-debug
.
For win32, this means adding:
/Zi /DEBUG /RTC1 /GS
To the compile phase, and:
/DEBUG
To the linker.
For linux:
-g -ggdb
is probably enough, see Debugging Options(gcc.gnu.org)
Change History (8)
comment:2 Changed 8 years ago by
Owner: | changed from Antoine Martin to ahuillet |
---|---|
Status: | new → assigned |
Some pointers:
- Cython debugging (cython docs)
- Cython debugging techniques (cython wiki)
- Debugging Cython Code on Windows X64 (not working here - but interesting nonetheless)
- r2951 added "
-g
" and "pyrex_gdb=True
" to the posix builds - to use it, callsetup.py
with "--with-debug
" - r2952 allows us to build with
--pyrex-gdb
option on win32 - you must setDEBUG=1
inwin32/MAKE-INSTALLER.BAT
Can you please try and let me know if that helps in debugging the cython code?
comment:3 Changed 8 years ago by
The changes do not seem to have any impact, in that the commandlines used for building do not include /DEBUG nor any of the options I mention in the ticket.
Adding those options manually doesn't work, per comment:1
comment:4 Changed 8 years ago by
ahuillet: please see the links in comment:2, we cannot just add /DEBUG
for cython code
comment:5 Changed 8 years ago by
comment:6 Changed 8 years ago by
Owner: | changed from ahuillet to Antoine Martin |
---|---|
Status: | assigned → new |
I don't know what to do about this - close if you feel like it. I do believe it can be useful to have debug symbols in the cython-produced libs, but if that's not feasible we have no choice.
comment:7 Changed 7 years ago by
Description: | modified (diff) |
---|---|
Resolution: | → fixed |
Status: | new → closed |
done in r5022 - not sure how useful this is going to be as we rarely ever get any crashes nowadays, but it's there.
Just build with:
./setup.py --with-debug
comment:8 Changed 6 weeks ago by
this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/291
The following patch does it: (broken link removed by totaam)
but it breaks because when building gdk_atoms, py2exe adds by default /Ox and this is incompatible with the options I have added