kicad-developers team mailing list archive

[Maciej Sumiński <maciej.suminski@xxxxxxx>]

Hi Mathias,

Thank you for the patch, I have just committed it.

Regards,
Orson

On 04/29/2017 09:35 PM, Mathias Grimmberger wrote:
> 
> Hi everybody,
> 
> 
> Currently KiCad doesn't build on OpenSuSE 42.2 because kicad-ogltest
> fails to compile.
> 
> The reason seems to be that wxWidgets was built with wxDEBUG_LEVEL 0,
> and because of that the global variable wxTheAssertHandler doesn't
> exist.
> 
> The fix is trivial, the function wxSetAssertHandler() exists regardless
> of the debug level, it just doesn't do anything at level 0.
> 
> See the tiny patch below,
> 
> 
> MGri
> 
> 
> --- kicad-ogltest.cpp.org	2017-04-29 21:01:11.444771665 +0200
> +++ kicad-ogltest.cpp	2017-04-29 21:03:17.520771387 +0200
> @@ -65,15 +65,14 @@
>      WX_QUIET()
>      {
>          m_old_level = wxLog::GetLogLevel();
> -        m_old_handler = wxTheAssertHandler;
> -        wxTheAssertHandler = nullptr;
> +        m_old_handler = wxSetAssertHandler( nullptr );
>          wxLog::SetLogLevel( wxLOG_FatalError );
>      }
>  
>      ~WX_QUIET()
>      {
>          wxLog::SetLogLevel( m_old_level );
> -        wxTheAssertHandler = m_old_handler;
> +        wxSetAssertHandler( m_old_handler );
>      }
>  };
> 
> 
> _______________________________________________
> 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
>

Attachment: signature.asc
Description: OpenPGP digital signature