Bullet 2.89 -> Bullet 3.17

[sloretz]

ros2/choco-packages#18

which is in the latest release: https://github.com/ros2/choco-packages/releases/tag/2022-03-15

[clalancette]

So this change for Rolling, as-is, is fine.

But it does make me wonder what our advice is for Foxy and Galactic users. Should they still download these latest packages? Or should we have them download the previous bullet 2.89.0?

[nuclearsandwich]

So this change for Rolling, as-is, is fine.

But it does make me wonder what our advice is for Foxy and Galactic users. Should they still download these latest packages? Or should we have them download the previous bullet 2.89.0?

Assuming that CI passes when ros-infrastructure/ros2-cookbooks#42 is applied to ros2/ci and we run extra CI to cover Foxy and Galactic, I think we want our dev instructions to do what CI does and use the latest version. We don't have any kind of update guide / upgrade strategy for ROS 2 on Windows. Everything assumes a from-scratch setup but according to sloretz here the currently packaged version of bullet is fine for all ROS 2 use cases.

If we do want to deviate on Windows, ros-infrastructure/ros2-cookbooks#42 will need to be updated to conditionally change packages per rosdistro.

[sloretz]

So this change for Rolling, as-is, is fine.

But it does make me wonder what our advice is for Foxy and Galactic users. Should they still download these latest packages? Or should we have them download the previous bullet 2.89.0?

Assuming that CI passes when ros-infrastructure/ros2-cookbooks#42 is applied to ros2/ci and we run extra CI to cover Foxy and Galactic, I think we want our dev instructions to do what CI does and use the latest version. We don't have any kind of update guide / upgrade strategy for ROS 2 on Windows. Everything assumes a from-scratch setup but according to ros2/choco-packages#18 (comment) the currently packaged version of bullet is fine for all ROS 2 use cases.

CI passed for Rolling, Galactic, and Foxy in ros-infrastructure/ros2-cookbooks#42 , so it and this one are ready to be merged. The documentation PR should be backported to Foxy and Galactic.