kicad-developers team mailing list archive

[Wayne Stambaugh <stambaughw@xxxxxxxxx>]

On 8/5/2015 10:16 AM, Maciej Sumiński wrote:
> Chris has pointed out that it was already discussed [1] and the change
> was not entirely clear. Apparently it had not been wrapped before
> revision 5834, so maybe OnPgmExit() should be still called for both
> Python and non-Python versions of pcbnew?

This was done because the problem did not occur when wxPython scripting
was disabled and Dick is planning on add wxPython on top of Kicad rather
than a bolted onto the side like our current implementation.
Compiling OnPgmExit() out of non wxPython builds was at his request.
The original crash had to do with wxPython clean up not being called
properly.  I just tried to duplicate this bug on a 32 bit build on
windows by running 'pcbnew foo` with no scripting enabled, and I cannot
get it to crash.  Are you testing this on Linux?

> 
> Regards,
> Orson
> 
> 1.
> https://www.mail-archive.com/kicad-developers@xxxxxxxxxxxxxxxxxxx/msg14014.html
> 
> On 08/05/2015 11:22 AM, Maciej Sumiński wrote:
>> Pcbnew without scripting support crashes when run with an invalid file
>> given as a parameter (stack overflow in ~wxSingleInstanceChecker()).
>>
>> There is a fix for that, but wrapped with #ifdef
>> KICAD_SCRIPTING_WXPYTHON .. #endif. I removed the mentioned directives
>> and it seems fine here, but I am not confident enough to commit it
>> immediately. Any thoughts?
>>
>> Regards,
>> Orson
>>
>>
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help   : https://help.launchpad.net/ListHelp
>>
> 
> 
> 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>