Uh oh!
There was an error while loading. Please reload this page.
ENH: Improve Qt include path resolution macOS - #271
Conversation
Add specific framework path handling Refactor include directory parsing to support multiple candidate paths for headers and introduce macOS-specific logic for resolving framework-relative header paths. Enhance generator flexibility and robustness for cross-platform usage. Push upstream fix via: cppcheck-opensource/simplecpp#448
| for (const QString &includeDir: getIncludeDirectories(commandLineIncludes)) | ||
| { | ||
| QFileInfo fi(QDir(includeDir), "qtcoreversion.h"); | ||
| std::list<std::string> candiate_paths; |
| std::string get_apple_framework_relative_path(const std::string& header); | ||
There was a problem hiding this comment.
Does this have to be public, or could you just put it into an anonymous namespace in the implementation?
| } | ||
| #ifdef __APPLE__ | ||
| static std::string get_apple_framework_relative_path(const std::string& header) |
There was a problem hiding this comment.
Please try to use the same naming style (e.g. camel case) as the rest of the code.
hjmjohnson
commented
Jul 17, 2025
I am working with the upstream simplecpp developers to finalize the changes. cppcheck-opensource/simplecpp#448 I'll make fixes there. It will probably be a few months before I get back to this. |
mrbean-bremen
commented
Jul 17, 2025
Ah, I had missed that this is from an upstream change. In this case it is of course preferrable to leave the code compatible, though it would be even better if you could change this upstream, thanks! |
jcfr
commented
Sep 19, 2025
@hjmjohnson This pull request can now be closed as it is superseded by the following pull request: 🙏 For the initial draft & investigative work, this was instrumental to understand the issues. |
mrbean-bremen
commented
Sep 19, 2025
Superseded by #302 |
Add specific framework path handling
Refactor include directory parsing to support multiple candidate paths for headers and introduce macOS-specific logic for resolving framework-relative header paths. Enhance generator flexibility and robustness for cross-platform usage.
Push upstream fix via:
cppcheck-opensource/simplecpp#448