Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make bindings_python error 127: Missing dependency in docs. #1238

Closed
santiagorg2401 opened this issue Feb 25, 2023 · 2 comments · Fixed by #1241
Closed

Make bindings_python error 127: Missing dependency in docs. #1238

santiagorg2401 opened this issue Feb 25, 2023 · 2 comments · Fixed by #1241

Comments

@santiagorg2401
Copy link

Hi,
I'm currently working on Ubuntu 22 and ROS2 Humble with Crazyswarm2, however, I've been facing an error while trying to build python bindings for Crazyswarm 2, after executing:

make cf2_defconfig
make bindings_python

I get:

Makefile:219: warning: overriding recipe for target 'bindings_python'
tools/kbuild/Makefile.kbuild:147: warning: ignoring old recipe for target 'bindings_python'
make[1]: Entering directory '/home/santiagorg2401/ros2_ws/crazyflie-firmware/build'
  GEN     ./Makefile
scripts/kconfig/conf  --silentoldconfig Kconfig
make[1]: Nothing to be done for 'bindings_python'.
make[1]: Leaving directory '/home/santiagorg2401/ros2_ws/crazyflie-firmware/build'
swig -python -Isrc/modules/interface -Isrc/hal/interface -Isrc/utils/interface -o build/cffirmware_wrap.c bindings/cffirmware.i
make: swig: No such file or directory
make: *** [Makefile:219: bindings_python] Error 127

So I installed SWIG by doing:
sudo apt -y install swig

And now it seems to be working.

@krichardsson
Copy link
Contributor

Thanks for the feedback!
I makes sense that swig is required...

@knmcguire knmcguire linked a pull request Mar 3, 2023 that will close this issue
@knmcguire
Copy link
Contributor

This should be fixed the next time the website is deployed

@knmcguire knmcguire added this to the 2023.07 milestone Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants