Skip to content

Debug sifdecoder_standalone segfault on macOS#39

Closed
jfowkes wants to merge 1 commit into
masterfrom
debug-segfault
Closed

Debug sifdecoder_standalone segfault on macOS#39
jfowkes wants to merge 1 commit into
masterfrom
debug-segfault

Conversation

@jfowkes

@jfowkes jfowkes commented Sep 8, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@jfowkes jfowkes changed the title Debug sifdecoder_standalone segfault on maCOS Debug sifdecoder_standalone segfault on macOS Sep 8, 2025
shell: bash
run: |
meson setup builddir -Ddefault_library=static
meson setup builddir -Ddefault_library=static --buildtype=debug

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FERRISDC doesn't ship with SIFDecode. Try downloading it and adding the offending command to sifdecoder.yml.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ignore my comment; I didn't realize that the entire MASTSIF was downloaded in the yml workflow.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah Alexis added a test of the entire MASTSIF collection.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... which, of course, grows from time to time as I add new examples :)

@jfowkes

jfowkes commented Sep 8, 2025

Copy link
Copy Markdown
Contributor Author

Closing as compilation in debug mode has now been merged into master.

@jfowkes jfowkes closed this Sep 8, 2025
@amontoison amontoison deleted the debug-segfault branch September 13, 2025 13:56
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

Successfully merging this pull request may close these issues.

3 participants