Bug 203210

Summary: PATCH: add volume control to playmus interactive mode
Product: [Fedora] Fedora Reporter: Hans de Goede <hdegoede>
Component: SDL_mixerAssignee: Brian Pepple <bdpepple>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: extras-qa
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 1.2.7-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-08-26 04:20:02 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
PATCH: add volume control to playmus interactive mode none

Description Hans de Goede 2006-08-18 21:52:24 UTC
Hi,

I've been working on getting d1x (descent 3d-shooter linux port) to play to
music from the original game. Now, 2 full days further I'm getting there.

Because of <insert long story here if you really want to know ask> I cannot use
SDL_mixer directly from d1x, so I'm using popen("playmus -i xxxx", "w") to start
playmus todo the playback and I control it (-i is interactive mode) by writing
the nescesarry commands to the stdin FILE * returned by popen.

However by default playmus' interactive mode doesn't allow changing the volume,
which I need to be able todo. The attached patch adds this capability to
playmus, and is otherwise harmless. I hope you can incorperate this patch into
playmus, so that I don't have to ship a modified version as part of the d1x package.

Thanks!


p.s.

d1x is in that other repo because it comes with a no commercial use allowed clause.

Comment 1 Hans de Goede 2006-08-18 21:52:25 UTC
Created attachment 134489 [details]
PATCH: add volume control to playmus interactive mode

Comment 2 Hans de Goede 2006-08-18 21:58:19 UTC
p.s.

I've send this patch upstream also I'll let you know their response.


Comment 3 Brian Pepple 2006-08-18 22:25:49 UTC
I'll try to give this a look over the weekend.

Comment 4 Brian Pepple 2006-08-26 04:20:02 UTC
Patch applied.  It will be available in SDL_mixer-1_2_7-1_fc6.