BrettRD / ros-gst-bridge

a bidirectional ros to gstreamer bridge and utilities for dynamic pipelines
Other
128 stars 30 forks source link

Expand on the build instructions; add a Dockerfile to help validate t… #28

Closed mutantbob closed 2 years ago

mutantbob commented 2 years ago

…he build instructions

This is how I would expand the Building section with a little more detail.

I have also included the Dockerfile I used to develop the instructions. Hopefully it can provide a little more detail for folks who need to get their environment set up.

BrettRD commented 2 years ago

This is great!

minor notes for the readme: In the first call to setup.bash, we're adding the system packages to the environment so rosdep doesn't try to install ros, and provide packages for cmake to start with. These deps are necessary but not sufficient to load the built plugin (missing audio_msgs package not yet built) Once the plugin (and audio_msgs) is built, ~/galactic_ws/install/setup.bash will be required to provide the audio_msgs .sos to load the plugin. ~/galactic_ws/install/setup.bash alone will work on a fresh shell because it'll refer back to the environment it was built against

Some notes for the Dockerfile:

I have changes to this effect in a local copy, could you re-target the pull request to the (newly created, even with branch ros2) build_instructions branch?

BrettRD commented 2 years ago

merged in https://github.com/BrettRD/ros-gst-bridge/pull/31