Conversation
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
fba7276 to
69ab5da
Compare
fc503f2 to
0a68da4
Compare
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
13a0469 to
4b0aa25
Compare
fujitatomoya
left a comment
There was a problem hiding this comment.
lgtm with green CI
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
Fixed RHEL failing |
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
|
@fujitatomoya Can you re-run the CI? It should be fixed now. |
Signed-off-by: Michael Carlstrom <rmc@carlstrom.com>
|
@fujitatomoya the Windows failure should be fixed. |
|
@fujitatomoya Should Windows still be building? |
|
Pulls: #1249 |
|
@sloretz @clalancette This PR caused regression in multiple tests in rosbag2 with the error message: You can see a full log from one of the failing jobs Curious if we can revert this PR? |
|
So we aren't seeing this failing in e.g. https://ci.ros2.org/view/nightly/job/nightly_linux_release/3164/ . My best guess is that the action there is using Ubuntu 22.04, instead of Ubuntu 24.04. Since we are only targeting Ubuntu 24.04, that action should probably be updated. |
Looks like the action is indeed running 22.04
|
|
Ok. Got it.
|
|
Is there a reason this can't be backported to jazzy? it would be nice to have the typedefs |
We've had a bunch of back and forth on these various PRs to get them in. Also, we've had ongoing CI issues with these changes (and while this is not the direct fault of these PRs, it is exacerbating the problem). Thus, I'm not comfortable backporting these, at least until we figure out what the CI issues are and what the rest of the PRs look like. |
No description provided.