← Back to team overview

kicad-developers team mailing list archive

Simple build fix for OpenSuSE 42.2

 

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 );
     }
 };



Follow ups