Skip to content

Full rebuild September 2026 bump ros2-distro-mutex to 0.19.0 and build number to 23 - #41

Open
mini-1235 wants to merge 103 commits into
RoboStack:mainfrom
mini-1235:update-lyrical-snapshot
Open

Full rebuild September 2026 bump ros2-distro-mutex to 0.19.0 and build number to 23#41
mini-1235 wants to merge 103 commits into
RoboStack:mainfrom
mini-1235:update-lyrical-snapshot

Conversation

@mini-1235

@mini-1235 mini-1235 commented Sep 2, 2026

Copy link
Copy Markdown

Supersedes #34 and #42

wolfv added 30 commits August 2, 2026 00:37
@sea-bass

sea-bass commented Sep 9, 2026

Copy link
Copy Markdown

I think bumping zenoh to 1.10.0 caused some issues with gazebo libraries; even on rolling (and even on the PR linked above) we seem to be still using 1.9.0. Reverting this to see how it goes.

If you had a reason to bump zenoh up that I'm unaware of, we can seek alternative solutions.

@mini-1235

Copy link
Copy Markdown
Author

If you had a reason to bump zenoh up that I'm unaware of, we can seek alternative solutions.

Reverting is fine with me. I originally bumped it to 1.10.0 because I wanted to test some of the new features, but I forgot that it could cause issues with Gazebo. My bad.

@sea-bass sea-bass changed the title Update lyrical snapshot Full rebuild September 2026 bump ros2-distro-mutex to 0.19.0 and build number to 23 Sep 9, 2026
@mini-1235

Copy link
Copy Markdown
Author

Should we include #42 in this PR?

@sea-bass

sea-bass commented Sep 9, 2026

Copy link
Copy Markdown

Should we include #42 in this PR?

Seems like a good idea. I guess I can just merge that branch in here and temporarily kill the caches for ros2cli and rosidl_cli?

Comment thread pkg_additional_info.yaml Outdated
sea-bass and others added 2 commits September 9, 2026 11:52
Co-authored-by: Maurice Alexander Purnawan <mauricepurnawan@gmail.com>
Comment thread robostack.yaml Outdated
robostack:
linux: [libuuid]
osx: []
osx: [libuuid]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this? MacOS has a native uuid implementation that I think is preferred. I remember something in the distant past about having issues with libuuid on osx.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mini-1235 can maybe answer that; I see in rolling this is not in there.

For my own education, would any cache need to be deleted if we revert this change?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I was trying to remove this from #34:
https://github.com/wolfv/ros-lyrical/blob/0f9db906590d2880b4bef68c8e6fa78da017ecb4/patch/dependencies.yaml#L100-L103
since libuuid isn’t available on all platforms. But looking at it again, this might not actually be necessary. Let me temporarily remove nav2_map_server and see if the build succeeds.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems like it's good -- nice!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If reverted, perhaps append a comment. To prevent the same in the future.

mini-1235 and others added 3 commits September 10, 2026 07:30
Signed-off-by: Maurice <mauricepurnawan@gmail.com>
Signed-off-by: Maurice <mauricepurnawan@gmail.com>
@sea-bass
sea-bass force-pushed the update-lyrical-snapshot branch from 00abcfa to 6127aa9 Compare September 10, 2026 01:01
@sea-bass

sea-bass commented Sep 10, 2026

Copy link
Copy Markdown

So MoveIt Task Constructor released a 0.2.0 version just last week with this change, and that's causing a bit of a stir-up on Windows as well.

Making some changes now, and these will likely apply to other distros whenever we do rebuilds there. yay.

Upstream PR in moveit/moveit_task_constructor#761

@sea-bass
sea-bass marked this pull request as ready for review September 10, 2026 05:54
@sea-bass

sea-bass commented Sep 10, 2026

Copy link
Copy Markdown

Build succeeded, but I got this error I'm unsure of on the Windows run at the very end:

RuntimeError: Generated workflow win.yml is 693.1 KiB, exceeding the 500 KiB limit. Increase --batch_size to reduce the number of generated jobs.

https://github.com/RoboStack/ros-lyrical/actions/runs/34427755718/job/102716901102?pr=41

I guess related to RoboStack/robostack.github.io#105?

@mini-1235

Copy link
Copy Markdown
Author

Maybe related to RoboStack/ros-rolling#34

I don't have my laptop now and I can't edit on my phone 😢

@sea-bass
sea-bass force-pushed the update-lyrical-snapshot branch from 8f2050c to 57d9927 Compare September 10, 2026 06:08
@sea-bass

Copy link
Copy Markdown

Maybe related to RoboStack/ros-rolling#34

Thank you! It is related, but seemingly this is the first time it's needed to be done in testpr.yml workflow rather than main.yml? I also posted on Discord to kick off that discussion.

But, we're all green here! Will let one of the maintainers give us the final go-ahead here.

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.

6 participants