Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Segfault running mull-reporter following the CTest example in the docs #1086

Closed
tvoskui opened this issue Feb 26, 2025 · 4 comments
Closed

Comments

@tvoskui
Copy link
Contributor

tvoskui commented Feb 26, 2025

I'm attempting to run multiple instances of mull-runner and combine the results using mull-reporter, following the example at

https://mull.readthedocs.io/en/0.24.0/tutorials/CTestIntegration.html

but I get a segfault when invoking mull-reporter at the final step. In RawReport mull::SQLiteReporter::loadRawReport in SQLiteReporter.cpp it appears that the code inside the following loop

while (sqlite3_step(selectMutantsStmt) == SQLITE_ROW) {

is using the wrong SQLite object (should be using selectMutantsStmt, but it's using selectInfoStmt in the calls to read the columns). I verified that changing those calls fixed my issue. I will create a PR to go along with this.

@AlexDenisov
Copy link
Member

Fixed via #1087, thank you for the report and for the fix @tvoskui, highly appreciated!

@AlexDenisov
Copy link
Member

I just cut a new release with the fix included https://github.com/mull-project/mull/releases/tag/0.25.0

@tvoskui
Copy link
Contributor Author

tvoskui commented Mar 2, 2025

Thanks for getting that in! I have been maintaining a Mull spack package internally and will add that version to the list. At some point I may want to push that to the main spack repo as an official package. I could add a section to the Mull docs on how to install and build with Mull and spack if you're interested.

https://github.com/spack/spack

@AlexDenisov
Copy link
Member

Thanks for getting that in! I have been maintaining a Mull spack package internally and will add that version to the list. At some point I may want to push that to the main spack repo as an official package. I could add a section to the Mull docs on how to install and build with Mull and spack if you're interested.

https://github.com/spack/spack

Thank you so much for doing that, a doc would be much appreciated!
Please, let me know if I can help with it somehow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants