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

misc: bump image_transport_plugins to 1.15.0-1 #494

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

daenny
Copy link

@daenny daenny commented Aug 26, 2024

I am not completely sure if this is the correct way, but ros/rosdistro#42174 was merged to the ros-distro. So I would like bump image transport plugins on robostack-staging to 1.15.0.

@daenny
Copy link
Author

daenny commented Sep 11, 2024

Bump, do I need to do anything else to get these published?

@Tobias-Fischer
Copy link
Collaborator

Sorry I’ve been really busy with teaching and paper/grant writing.

I think we shouldn’t bump individual packages for reasons outlined in RoboStack/ros-humble#204

We should instead aim for a full rebuild of all packages.

@daenny
Copy link
Author

daenny commented Sep 11, 2024

Okay, is there any process to do that? The problem at the moment is, that the image_transport_plugins is broken on ros-distro-mutex 0.5 due to an opencv update.

@traversaro
Copy link
Member

Okay, is there any process to do that? The problem at the moment is, that the image_transport_plugins is broken on ros-distro-mutex 0.5 due to an opencv update.

In which sense? As a temporary solution have you considered pinning opencv to a working version?

@daenny
Copy link
Author

daenny commented Sep 11, 2024

ros/rosdistro#42149

And:
ros-perception/image_transport_plugins#130

ros-distro-mutex 0.5 pins opencv to 4.9. This introduced strict assert check on params.size() in various functions. There is AFAIK no other way releasing the correct version of image_transport_plugins as opencv is pinned by the mutex. Currently we have released the patched version on an internal quetz server.

@traversaro
Copy link
Member

traversaro commented Sep 11, 2024

Thanks, now the situation is clear to me.

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.

3 participants