Bump to the latest commit on ros2_tracing to fix deprecation.#899
Bump to the latest commit on ros2_tracing to fix deprecation.#899nuclearsandwich merged 1 commit intomasterfrom
Conversation
Since the release of ros2_tracing including this change is delayed we'll temporarily pin the commit which addresses the deprecation warning. Signed-off-by: Steven! Ragnarök <steven@nuclearsandwich.com>
|
The windows build hung during tests (ros2/build_farmer#248) but only after completing a full build. Security failures with Connext on macOS are related to ros2/system_tests#409 |
|
Why is the version a commit hash instead of the |
As @nuclearsandwich mentioned, I have to delay the Foxy release. I want to include some more things for the quality declaration, but I personally haven't had time yet (I blame school work 😛)
Up until now we've always used release tags instead of branches to properly test I'd argue that it's not "unstable" and definitely wouldn't mind switching to branches, but I can't run CI on all supported platforms, so that's that. |
|
Hmmm... Even if it's not release-ready, I'd still rather use |
|
Before we used a tag because we don't want to track master and be exposed to any changes landing there and potentially regressing the build. That rational hasn't changed. |
|
Sure, if master is unsuitably unstable that it may regress the build, can we maybe create a new ros2_tracing branch for ros2.repos to track that is slower moving so as not to regress the build? Your rationale applies to every package in ros2.repos. |
The significant difference that we don't have write ccess to
Upstream is planning to tag a new version soon (https://gitlab.com/micro-ROS/ros_tracing/ros2_tracing/-/merge_requests/163#note_321456164) and then we will use that tag (as we have done before). |
|
Since the release of ros2_tracing including this change is delayed we'll
temporarily pin the commit which addresses the deprecation warning.
There are a substantial number of changes since the last release (0.3.0...master) so this may not go down easy.
This PR "causes" #898 which will be resolved when we're back on a release channel.
Signed-off-by: Steven! Ragnarök steven@nuclearsandwich.com