[Buildroot] [Bug 477] New: Add sdl_sound package

[Buildroot] [Bug 477] New: Add sdl_sound package

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=477

           Summary: Add sdl_sound package
           Product: buildroot
           Version: unspecified
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: enhancement
          Priority: P5
         Component: Other
        AssignedTo: unassigned at buildroot.uclibc.org
        ReportedBy: evan.zelkowitz at gmail.com
                CC: buildroot at uclibc.org
   Estimated Hours: 0.0


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


-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 477] Add sdl_sound package

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=477


Peter Korsgaard <jacmet@uclibc.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED




--- Comment #1 from Peter Korsgaard <jacmet@uclibc.org>  2009-07-22 07:18:12 UTC ---
Sure, please send a patch.


-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 477] Add sdl_sound package

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=477





--- Comment #2 from Evan Zelkowitz <evan.zelkowitz@gmail.com>  2009-07-28 04:05:13 UTC ---
Created an attachment (id=505)
 --> (https://bugs.busybox.net/attachment.cgi?id=505)
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?


-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 477] Add sdl_sound package

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=477





--- Comment #3 from Evan Zelkowitz <evan.zelkowitz@gmail.com>  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


-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 477] Add sdl_sound package

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=477


Evan Zelkowitz <evan.zelkowitz@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #505 is|0                           |1
           obsolete|                            |




--- Comment #4 from Evan Zelkowitz <evan.zelkowitz@gmail.com>  2009-07-29 23:13:21 UTC ---
Created an attachment (id=515)
 --> (https://bugs.busybox.net/attachment.cgi?id=515)
patch to add sdl sound

should replace previous patch since this one used git diff


-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 477] Add sdl_sound package

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=477


Peter Korsgaard <jacmet@uclibc.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED




--- Comment #5 from Peter Korsgaard <jacmet@uclibc.org>  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


-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.