Skip to content
This repository has been archived by the owner on Jan 14, 2023. It is now read-only.

Add audio_common_msgs and sound_play #26

Open
wants to merge 6 commits into
base: indigo
Choose a base branch
from

Conversation

compiaffe
Copy link

Added dependencies to
audio_common_msgs and sound_play

@stonier
Copy link
Contributor

stonier commented Dec 12, 2016

@adamantivm this is a reasonable addition we should get in for kinetic. I don't like adding sound_play here though. It has code in that package - better if the msg in sound_play was actually in audio_common_msgs. We should push that as a change upstream.

If we do add sound_play, then test it first to make sure the generating code doesn't break.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants