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

Kirkstone fixes #1009

Closed
wants to merge 4 commits into from
Closed

Kirkstone fixes #1009

wants to merge 4 commits into from

Conversation

moto-timo
Copy link
Contributor

This series mostly fixes deprecated distutils usage, but also changes the release compatibility from 'honister' to 'kirkstone'

a0e7adc ros_catkin.bbclass: inherit setuptools3-base
876a5e2 python3-pydot: inherit setuptools3
d0a89e1 python3-empy: patch to use setuptools3
f2d275d ros-distro.conf: ROS_OE_RELEASE_SERIES -> kirkstone

Upstream is inactive, but it is a simple enough patch to switch from
distutils.core to setuptools so we can inherit setuptools3 and silence
the distutils3 deprecation warning.

Signed-off-by: Tim Orling <[email protected]>
setup.py actually tests for setuptools first and falls back on
distutils. Silence the distutils3 deprecation warning by inheriting
setuptools3 instead.

Signed-off-by: Tim Orling <[email protected]>
Replace deprecated distutils3-base with setuptools3-base

Signed-off-by: Tim Orling <[email protected]>
@moto-timo
Copy link
Contributor Author

Not asking for this to be merged, but just FYI: https://github.com/moto-timo/meta-ros/tree/superflore-kirkstone-2022-04
This was only running superflore and no attempt was made to build or test.

@ros-discourse
Copy link

This pull request has been mentioned on ROS Discourse. There might be relevant details there:

https://discourse.ros.org/t/announcing-the-completion-of-meta-ros-milestone-17/25933/3

@ros-discourse
Copy link

This pull request has been mentioned on ROS Discourse. There might be relevant details there:

https://discourse.ros.org/t/ros-2-tsc-meeting-minutes-2023-01-19/29423/7

@DRK-512
Copy link

DRK-512 commented Sep 28, 2023

I pulled your kirkstone-fixes branch, and I get the following error when running it:
"Variable PNBLACKLIST has been renamed to SKIP_RECIPE (file: layers/meta-ros/meta-ros-python2/recipes-extended/wxwidgets/wxpython_3.0.2.0.bb line: 45)1"

I made a PR that is the same as yours, except more up-to-date and is based off the langdale branch as opposed to the honister one. It is PR #1051, feel free to check it out

@robwoolley
Copy link
Collaborator

Thanks, @moto-timo

I have pulled in your fixes into the kirkstone commit series I am getting ready to push before Monday.

@robwoolley robwoolley closed this Nov 10, 2023
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.

4 participants