← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Crash Reporter

 

Hey Tom,

On 5/29/2019 9:56 AM, Tomasz Wlostowski wrote:
> On 29/05/2019 06:07, Seth Hillbrand wrote:
>> On 2019-05-28 12:13, Tomasz Wlostowski wrote:
>>>
>>> Done (tom-crash-reporter-may27 on my Github)!
>>>
>>> Tom
>>
>> Hi Tom-
>>
>> This looks very nice.  I like the implementation a lot!
>>
>> Here are a couple minor bits:
>>
>> 1) The force crash dialog says "Clicking on OK will crash KiCad" but the
>> buttons say "Yes/No"
>> 2) The crash report doesn't seem to pick up my build version
>> information.  It says:
>>
>> KiCad crash report, version 1.0
>> --------------------------------------------------
>>
>> application: kicad
>> version:     , release build
>>
>> 3) The tabbing changes in CMakeLists.txt are odd.  Intentional?
>> 4) There's an untranslated string in OnSendReport ("Error sending cebug
>> report.")[sic]
>>
> Hi Seth,
> 
> Just pushed a fix for these. Thanks for testing!
> 
> Tom
> 

I had no luck building this on windows both 32 and 64 bit builds.  I'm
getting two build errors:

The first build error is not related to the debug reporting but this
error (and warning) is:

E:/msys64/home/Wayne/src/kicad-twlostow/common/debug_report.cpp: In
function 'const wxString formatHex(uint64_t)':
E:/msys64/home/Wayne/src/kicad-twlostow/common/debug_report.cpp:150:27:
warning: format '%p' expects argument of type 'void*', but argument 4
has type 'size_t' {aka 'unsigned int'} [-Wformat=]
  150 |     snprintf(tmp, 1024, "%p", (size_t) addr);
      |                          ~^   ~~~~~~~~~~~~~
      |                           |   |
      |                           |   size_t {aka unsigned int}
      |                           void*
      |                          %d
E:/msys64/home/Wayne/src/kicad-twlostow/common/debug_report.cpp: In
member function 'void DEBUG_REPORT::buildVersionInfo(wxString&)':
E:/msys64/home/Wayne/src/kicad-twlostow/common/debug_report.cpp:277:61:
error: 'OCC_VERSION_COMPLETE' was not declared in this scope
  277 |     aMsg << indent4 << "opencascade-community-edition: " <<
OCC_VERSION_COMPLETE << eol;
      |
^~~~~~~~~~~~~~~~~~~~
E:/msys64/home/Wayne/src/kicad-twlostow/common/debug_report.cpp: In
member function 'bool DEBUG_REPORT::AddContext(wxDebugReport::Context)':
E:/msys64/home/Wayne/src/kicad-twlostow/common/debug_report.cpp:481:44:
error: expected '}' at end of input
  481 |     buildExceptionContextInfo( exceptInfo );
      |                                            ^
E:/msys64/home/Wayne/src/kicad-twlostow/common/debug_report.cpp:469:1:
note: to match this '{'
  469 | {
      | ^
E:/msys64/home/Wayne/src/kicad-twlostow/common/debug_report.cpp:481:44:
warning: no return statement in function returning non-void [-Wreturn-type]
  481 |     buildExceptionContextInfo( exceptInfo );
      |                                            ^
make[2]: *** [common/CMakeFiles/common.dir/build.make:1214:
common/CMakeFiles/common.dir/debug_report.cpp.obj] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/Makefile2:264:
common/CMakeFiles/common.dir/all] Error 2
make: *** [Makefile:141: all] Error 2


Follow ups

References