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

{ros2} Add required ROS2 SDK environment variables #1261

Draft
wants to merge 1 commit into
base: master-next
Choose a base branch
from

Conversation

sgstreet
Copy link
Contributor

@sgstreet sgstreet commented Nov 20, 2024

This PR adds the following environment variable when the SDK is setup:

PYTHON_SOABI required to avoid setup failures with python packages
PYTHON3_NUMPY_INCLUDE_DIR required to using python opencv binding
PYTHONWARNINGS to suppress setuptool warmings
AMENT_SKIP_SHELL_PATH to prevent ament from prepending $OECORE_TARGET_SYSROOT/opt/ros/$ROS_DISTRO/bin to glocal PATH variable
PYTHONPATH to include the target site-packages to the python environment

If the Yocto enviroment variable ROS_SDK_UNIFY is not empty, $OECORE_TARGET_SYSROOT/opt/ros/$ROS_DISTRO/setup.bash or friends is source during SDK setup. The value of ROS_SDK_UNIFY should one of the following "sh", "bash" or "zsh"

@sgstreet sgstreet marked this pull request as draft November 25, 2024 22:13
@sgstreet
Copy link
Contributor Author

Still verifying this on master-next and styhead-next

@sgstreet
Copy link
Contributor Author

sgstreet commented Nov 25, 2024

@robwoolley I verify this by building examples, demos and example-interfaces with master-next and styhead-next. We still have the issue of PYTHON_SOABI verses _PYTHON_SYSCONFIGDATA_NAME. I have been unable to make the latter work.

@sgstreet
Copy link
Contributor Author

I have made a mess out of this, fixing now.

@sgstreet sgstreet force-pushed the add-env-vars-to-sdk branch from 7324827 to e16a14d Compare December 9, 2024 17:55
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