Skip to contentSkip to conversation

New: Add sdl_sound package

Buildroot bugzilla at busybox.net 1 participant 6 messages Jul 22, 2009 – Sep 20, 2009

lore
Share & tools
lore
#0 lore raw

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

Signature · 3 lines
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.
#1 lore raw

[Buildroot] [Bug 477] Add sdl_sound package

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

Peter Korsgaard <[email protected]> changed:

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

--- Comment #1 from Peter Korsgaard <[email protected]> 2009-07-22 07:18:12 UTC --- Sure, please send a patch.

Signature · 3 lines
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.
#2 lore raw

[Buildroot] [Bug 477] Add sdl_sound package

In reply to bugzilla at busybox.net

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

--- Comment #2 from Evan Zelkowitz <[email protected]> 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?

Signature · 3 lines
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.
#3 lore raw

[Buildroot] [Bug 477] Add sdl_sound package

In reply to bugzilla at busybox.net

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

--- Comment #3 from Evan Zelkowitz <[email protected]> 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

Signature · 3 lines
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.
#4 lore raw

[Buildroot] [Bug 477] Add sdl_sound package

In reply to bugzilla at busybox.net

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

Evan Zelkowitz <[email protected]> changed:

What |Removed |Added
----------------------------------------------------------------------------

 Attachment #505 is|0                           |1

obsolete| |

--- Comment #4 from Evan Zelkowitz <[email protected]> 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

Signature · 3 lines
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.
#5 lore raw

[Buildroot] [Bug 477] Add sdl_sound package

In reply to bugzilla at busybox.net

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

Peter Korsgaard <[email protected]> changed:

What |Removed |Added
----------------------------------------------------------------------------

             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #5 from Peter Korsgaard <[email protected]> 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

Signature · 3 lines
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.