Bug 477

Summary:	Add sdl_sound package
Product:	buildroot	Reporter:	Evan Zelkowitz <evan.zelkowitz>
Component:	Other	Assignee:	unassigned
Status:	RESOLVED FIXED
Severity:	enhancement	CC:	buildroot
Priority:	P5
Version:	unspecified
Target Milestone:	---
Hardware:	PC
OS:	Linux
Host:		Target:
Build:
Attachments:	sdl_snd patch patch to add sdl sound

Description Evan Zelkowitz 2009-07-22 06:36:54 UTC

SDL_sound is a library that handles the decoding of several popular sound file formats, such as .WAV and .MP3. It is meant to make the programmer's sound playback tasks simpler. The programmer gives SDL_sound a filename, or feeds it data directly from one of many sources, and then reads the decoded waveform data back at her leisure. If resource constraints are a concern, SDL_sound can process sound data in programmer-specified blocks. Alternately, SDL_sound can decode a whole sound file and hand back a single pointer to the whole waveform. SDL_sound can also handle sample rate, audio format, and channel conversion on-the-fly and behind-the-scenes, if the programmer desires. 

http://icculus.org/SDL_sound/

I have added this to my local buildroot so I figured I would see if this was desired on git

Comment 1 Peter Korsgaard 2009-07-22 07:18:12 UTC

Sure, please send a patch.

Comment 2 Evan Zelkowitz 2009-07-28 04:05:13 UTC

Created attachment 505 [details]
sdl_snd patch

Patch to add sdl_snd as an option under the sdl options in package selection.  Hopefully this patch is ok, I just did a diff against a fresh git and a fresh one with the new option.  It diffed a few of the binary git files, but since they were binary and just ended up as notifications in the patch I figured they were harmless.  Is there an official doc/faq/whatever about the patch submission process or any sort of standards for spacing on what not?

Comment 3 Evan Zelkowitz 2009-07-28 14:06:18 UTC

Just saw your response, was not aware there was a git diff program, Im new to git, usually use clearcase or svn.  You can either use this patch or I can redo it using git diff instead

Comment 4 Evan Zelkowitz 2009-07-29 23:13:21 UTC

Created attachment 515 [details]
patch to add sdl sound

should replace previous patch since this one used git diff

Comment 5 Peter Korsgaard 2009-09-20 08:07:00 UTC

Thanks, committed with minor changes:

 - sdl_snd -> sdl_sound
 - cleaned up help text
 - don't transform binary name
 - add option to keep playsound/playsound_simple - Off by default
 - fix uninstall when BR2_HAVE_DEVFILES isn't enabled