Skip to content

Conversation

PeterBowman
Copy link
Member

@PeterBowman PeterBowman commented Sep 9, 2025

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.:

cmake .. -DYARP_COMPILE_BINDINGS=ON -DCREATE_PYTHON=ON

The following target fails with the aforementioned error:

make yarp_python_swig_compilation

Using ${SWIG_YARP_LIBRARIES} instead of an explicit YARP::YARP_dev for maintainability, although this is the only YARP component needed.

Copy link

update-docs bot commented Sep 9, 2025

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would update the release notes by adding a file in doc/release/<target_branch>, based on your changes.

Copy link

@randaz81 randaz81 added this to the YARP 3.12.2 milestone Sep 12, 2025
@randaz81 randaz81 self-requested a review September 12, 2025 09:43
@randaz81 randaz81 merged commit 0548f6a into robotology:yarp-3.12 Sep 12, 2025
63 of 73 checks passed
@PeterBowman PeterBowman deleted the python-swig-depends branch September 12, 2025 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants