Skip to content

add point-cloud-transport-plugins#82

Open
ruben-arts wants to merge 4 commits into
mainfrom
add-point-cloud-transport-plugins
Open

add point-cloud-transport-plugins#82
ruben-arts wants to merge 4 commits into
mainfrom
add-point-cloud-transport-plugins

Conversation

@ruben-arts
Copy link
Copy Markdown
Contributor

@ruben-arts ruben-arts commented May 28, 2026

It doesn't work on windows in the other distros.

This also adds the diagnostic_remote_logging package to kilted.

@ruben-arts
Copy link
Copy Markdown
Contributor Author

@traversaro Do you have an idea why windows is trying to build the diagnostic-remote-logging It's a dependency of the packages I'm adding but even with that specific package added to the not win it still tries to build it.

@traversaro
Copy link
Copy Markdown
Member

@traversaro Do you have an idea why windows is trying to build the diagnostic-remote-logging It's a dependency of the packages I'm adding but even with that specific package added to the not win it still tries to build it.

There is a deep bug in vinca for which the wrong (i.e. the latest, instead of the snapshot) version of dependencies is used for several operations, see RoboStack/vinca#93 . I suspect what is happening is a recent update added diagnostic-remote-logging as a dependency for a package that is built. Clearly it would be ideal to fix this at the vinca level, but in this specific case we already fixed the problem in RoboStack/ros-jazzy#212 (comment), so I guess the easiest solution is add a patch here for windows and call it a day.

@ruben-arts ruben-arts requested a review from traversaro May 29, 2026 11:41
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.

2 participants