MetanoKid / cpp-build-analyzer

Analyzes MSVC C++ compilations with C++ Build Insights SDK
GNU General Public License v3.0
80 stars 2 forks source link

Crash on analysis #33

Closed ghost closed 4 years ago

ghost commented 4 years ago

With commit 53a50ab57f382089880d4a16ba64f6dacf75e743 CppBuildAnalyzer_20200528_195409.zip

MetanoKid commented 4 years ago

Hi @eric3ds!

Thank you for opening this issue and sharing the MiniDump! 😃 I'm afraid I'm still figuring out which is the best way for us to make investigations easier with these dumps.

Unfortunately, I haven't been able to extract helpful data from the MiniDump. Could you please try to reproduce the crash in the Debug configuration? MiniDumps from Debug configurations should include way more data to point us to the issue.

Thank you!

ghost commented 4 years ago

Here's the mini-dump generated while running the debug version of your tool CppBuildAnalyzer_20200528_233806.zip

MetanoKid commented 4 years ago

Hello, @eric3ds!

Thank you for providing the mini-dump in Debug, you were very kind!

Please let me apologize because the report process isn't as polished as I wanted it to be. You're its first real tester!

I've learned that mini-dumps require the exact .exe and .pdb files that generated them to be able to debug. I thought I could use my own files knowing the commit hash, but it turns out that's only useful to inspect the code from the .pdb related to the .dmp!

I've modified the report message to mention this for future users (49dc83197b1cc05f45b556b417938bfdd266352e).

I can still investigate your crash if you could attach matching .dmp, .exe and .pdb files :)

Thanks for your time, and sorry again!

ghost commented 4 years ago

I zipped the ConsoleMain folder and attached ConsoleMain.zip

MetanoKid commented 4 years ago

Thank you, @eric3ds!

I've managed to properly load the .dmp and will be working on a fix.

MetanoKid commented 4 years ago

I've started investigating and it turns out the issue comes from an invalid command-line option. It should be --analyze_all but it was invoked with --analyze-all (note the hyphen/underscore difference).

The command-line parsing library we use throws an exception when an invalid option is found, so we'll work on capturing those exceptions and hinting for valid ones.

I wanted to let you know so you can start getting useful data for your build while I work on improving this scenario.

Thank you!

ghost commented 4 years ago

Thanks. The analysis completed.

ghost commented 4 years ago

What graphing tool did you use for the bar charts?

[cid:image001.png@01D63B50.032B7350] -Eric

From: Carlos [mailto:notifications@github.com] Sent: Saturday, May 30, 2020 9:18 AM To: MetanoKid/cpp-build-analyzer Cc: HASAN Eric; Mention Subject: Re: [MetanoKid/cpp-build-analyzer] Crash on analysis (#33)

I've started investigating and it turns out the issue comes from an invalid command-line option. It should be --analyze_all but it was invoked with --analyze-all (note the hyphen/underscore difference).

The command-line parsing library we use throws an exception when an invalid option is found, so we'll work on capturing those exceptions and hinting for valid ones.

I wanted to let you know so you can start getting useful data for your build while I work on improving this scenario.

Thank you!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/MetanoKid/cpp-build-analyzer/issues/33#issuecomment-636329772, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALPMZTGKALNJWSMCN5D2OW3RUEBPPANCNFSM4NNMMSXQ.

This email and any attachments are intended solely for the use of the individual or entity to whom it is addressed and may be confidential and/or privileged.

If you are not one of the named recipients or have received this email in error,

(i) you should not read, disclose, or copy it,

(ii) please notify sender of your receipt by reply email and delete this email and all attachments,

(iii) Dassault Systèmes does not accept or assume any liability or responsibility for any use of or reliance on this email.

Please be informed that your personal data are processed according to our data privacy policy as described on our website. Should you have any questions related to personal data protection, please contact 3DS Data Protection Officer at 3DS.compliance-privacy@3ds.commailto:3DS.compliance-privacy@3ds.com

For other languages, go to https://www.3ds.com/terms/email-disclaimer

MetanoKid commented 4 years ago

It's Microsoft Power BI, but you could build those charts in any spreadsheet software that loads the .csv files as well.

ghost commented 4 years ago

Thanks. I want to use the same tools in case there are any issues that I need to report.

-Eric

On Jun 6, 2020, at 3:41 AM, Carlos notifications@github.com wrote:



It's Microsoft Power BIhttps://powerbi.microsoft.com/, but you could build those charts in any spreadsheet software that loads the .csv files as well.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/MetanoKid/cpp-build-analyzer/issues/33#issuecomment-640005449, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALPMZTAJYM7GDGIDLNIZNTDRVHXJ3ANCNFSM4NNMMSXQ.

This email and any attachments are intended solely for the use of the individual or entity to whom it is addressed and may be confidential and/or privileged.

If you are not one of the named recipients or have received this email in error,

(i) you should not read, disclose, or copy it,

(ii) please notify sender of your receipt by reply email and delete this email and all attachments,

(iii) Dassault Systèmes does not accept or assume any liability or responsibility for any use of or reliance on this email.

Please be informed that your personal data are processed according to our data privacy policy as described on our website. Should you have any questions related to personal data protection, please contact 3DS Data Protection Officer at 3DS.compliance-privacy@3ds.commailto:3DS.compliance-privacy@3ds.com

For other languages, go to https://www.3ds.com/terms/email-disclaimer

MetanoKid commented 4 years ago

Great, thanks for your support! 🙂