gazebo-tooling / release-tools

8 stars 9 forks source link

brew: pip install protobuf in CI #1127

Closed azeey closed 3 months ago

azeey commented 3 months ago

gz-transport Python tests are failing because Python bindings were removed in the latest update of protobuf homebrew bottle. It seems like the recommended way to install the Python bindings is through pip. Since this will be needed for anything above gz-transport, I thought we should just add it here to PIP_PACKAGES_NEEDED as a global dependency.

scpeters commented 3 months ago

Related:

I will experiment with putting protobuf into formula resource blocks, but I'm guessing it won't work for users who just want to run python3 and import and use our bindings

scpeters commented 3 months ago

testing:

scpeters commented 3 months ago

I think we're going to need to add pip install protobuf to our macOS installation instructions