Skip to content

fixed #14626 - execinfo.h detection in Makefile did not work#8401

Open
firewave wants to merge 1 commit intocppcheck-opensource:mainfrom
firewave:execinfo_h
Open

fixed #14626 - execinfo.h detection in Makefile did not work#8401
firewave wants to merge 1 commit intocppcheck-opensource:mainfrom
firewave:execinfo_h

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

@firewave firewave commented Apr 1, 2026

No description provided.

@firewave firewave marked this pull request as draft April 1, 2026 08:57
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 9, 2026

@firewave firewave force-pushed the execinfo_h branch 2 times, most recently from e46676d to 46bf0d2 Compare April 25, 2026 07:12
@firewave firewave marked this pull request as ready for review April 25, 2026 07:16
@firewave
Copy link
Copy Markdown
Collaborator Author

This turned out rather awful but when make fixed the handling of # in $(shell) they unfortunately did so in a way which is not backwards compatible.

There is a follow-up which improves the handling in config.h and will make the handling obsolete if we ever move to C++17 (please do not use this as a reason to do so - we should move to C++14 first - if we ever do so).

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.

1 participant