← Back to team overview

kicad-developers team mailing list archive

Re: Separated debug symbols

 

Le 26/10/2018 à 22:18, Maciej Suminski a écrit :
> What do you think about providing archives with debug symbols for
> Windows builds? It is a platform where we rarely get a stacktrace,
> because the process is not trivial and Windows debug builds take lots of
> space (_pcbnew.kiface is ~1.5 GB on my PC). Normally it would be wasted
> bandwidth, but from time to time we get bug reports which noone else can
> reproduce and then a stacktrace would be of great help.
> 
> I have written a patch that adds a CMake flag to split debug information
> to another file. This way we could offer compact binaries to the users
> and ask them to download debug information only in case of serious
> problems. There is already GDB shipped with the nightly installers, so
> then getting a stacktrace becomes feasible or we could use catchsegv [1]
> for an automated approach (see the attached example log; do not worry
> about the null pointer dereference - I added it temporarily to test the
> software). Obviously, Thomas' crash reporter looks even more convenient,
> but it is not mingw-gcc compatible [2]. The patch is also compatible
> with BUILD_SMALL_DEBUG_FILES flag.
> 
> I have not pushed the patch yet, as I still wonder whether adding
> another CMake flag is a good idea. Just now I realized that there are
> just a few extra commands to create the needed files, so perhaps they
> could be executed by the build scripts on Jenkins hosts.
> 
> Cheers,
> Orson
> 
> 1. https://github.com/jrfonseca/drmingw/tree/master/src/catchsegv
> 2.
> https://docs.wxwidgets.org/3.0/group__group__funcmacro__appinitterm.html#ga28a4fb827b93fa6bac18c9666c23ee10

Hi Orson,

I tested your patch without any issue on W7 32 bits and gdb.

With -DBUILD_SMALL_DEBUG_FILES=ON option, the binaries are about 20 time
smaller than without this option.

In fact on my 32 bits install, without this option, I cannot load pcbnew
from Kicad or with gdb and load a medium complexity board: I am running
out of memory.

In countries using a comma as FP separator, there is a problem:
When loading a bitmap, and perhaps some config files, a wxASSERT is
displayed (see attached copy)
I tried to fix this wxWidgets alert, but no luck.

-- 
Jean-Pierre CHARRAS

Attachment: wxwidgets_alert_fp_separator.png
Description: PNG image


Follow ups

References