Bug 234035

Summary: xine-lib 1.1.4-3 not compiled with pulseaudio
Product: [Fedora] Fedora Reporter: Jost Diederichs <jost>
Component: xine-libAssignee: Aurelien Bompard <gauret>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: ville.skytta
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 1.1.4-4 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-03-26 20:00:00 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 to xine-lib.spec none

Description Jost Diederichs 2007-03-26 18:36:12 UTC 
Description of problem:
while pulseaudio is included in the xine-lib sources and pulseaudio has been
included in fedora-devel for a while, pulseaudio is not enabled in the xine-lib
rpm spec.

Version-Release number of selected component (if applicable):
1.1.4-3

Additional info: 
a simple change to xine-lib.spec will add the pulseaudio plugin to
xine-lib-extras. I tried it, it compiles and works fine.
The patch is attached
Comment 1 Jost Diederichs 2007-03-26 18:36:14 UTC 
Created attachment 150926 [details]
patch to xine-lib.spec
Comment 2 Ville Skyttä 2007-03-26 19:08:06 UTC 
Hmm, is pulseaudio-devel a build dependency?  At least the *.so is built with
just pulseaudio-lib-devel there.

It seems that an unrelated libsmbclient-devel change snuck in the patch, looks
like that's something we'll need to take care of in FC7.
Comment 3 Jost Diederichs 2007-03-26 19:29:41 UTC 
You are correct,
it does build without pulseaudio-devel.

On the libsmbclient-devel requirement, I did sneek that in - the package
wouldn't build without it.
Comment 4 Ville Skyttä 2007-03-26 20:00:00 UTC 
Thanks for the confirmation, done in soon-to-appear 1.1.4-4, devel only at least
for now.