Fix rolling builds#323
Conversation
77f944b to
be6b0e5
Compare
b85c158 to
d2c43d8
Compare
|
I have just merged the first PR. Rerunning the CI now |
|
This PR is good to merge from my side. The current failures are related to embeddedRTPS, which I would say is out of the scope of this PR. After all current PRs are merged, I will make a PR to temporarily disable its builds in CI until we fix it/it gets updated to rolling. |
|
@Narukara @EugenioCollado could you review/merge this PR |
18fe3f2 to
1a34225
Compare
|
Sorry for the delay, @Narukara. I have explained the changes in detail in the comments added to |
|
LGTM
No problem at all — it's been the holidays lately. |
|
Hi @Amronos and @Narukara , based on the comment |
|
Thanks a lot @EugenioCollado! Looks like that worked! |
Some packages needed to be pinned to older versions, and
COLCON_IGNOREhad to be added to some new ones.Look at the PR diff for more info.
This is part 2 of 4 in a stack made with GitButler:
Changes specific to this PR:
d2c43d8