Generate dependent IDL headers before parsing yarp.i #3282
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR #3269 requires that the yarp/dev/LLM_Message.h header is present at the time yarp.i is parsed. If YARP bindings are built along with the core libraries, it can cause a "Error: Unable to find 'yarp/dev/LLM_Message.h'" SWIG compiler error due to this header being generated by the IDL parser afterwards.
To reproduce this, create a build tree from scratch and configure YARP bindings, e.g.:
The following target fails with the aforementioned error:
Using
${SWIG_YARP_LIBRARIES}
instead of an explicitYARP::YARP_dev
for maintainability, although this is the only YARP component needed.