Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 7397

Summary: incorrect api version defined in xine-d5d package
Product: Gentoo Linux Reporter: Charles Mauch <xterminus>
Component: Current packagesAssignee: Arcady Genkin (RETIRED) <agenkin>
Status: RESOLVED FIXED    
Severity: normal CC: gentoo3281, lostlogic
Priority: High    
Version: 1.2   
Hardware: x86   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: check for xine-lib api ver 8 as well on line 1053 of input_dvd.c
ebuild for previous patch
digest for previous ebuild

Description Charles Mauch 2002-09-02 21:25:39 UTC
please edit line 1053 of input_dvd.c in the src directory of the xine-d5d
package and change

  if (iface != 7) {

to

  if (iface != 8) {

Without changing this, this xine-d5d plugin is useless in newer versions of xine.
Comment 1 Joshua McKinney 2002-09-19 10:39:19 UTC
Created attachment 4010 [details, diff]
check for xine-lib api ver 8 as well on line 1053 of input_dvd.c

this seems to install and play fine, I haven't extenensively tested this
though.
Comment 2 Joshua McKinney 2002-09-19 10:40:10 UTC
Created attachment 4011 [details]
ebuild for previous patch
Comment 3 Joshua McKinney 2002-09-19 10:40:37 UTC
Created attachment 4012 [details]
digest for previous ebuild
Comment 4 Joshua McKinney 2002-09-19 11:01:02 UTC
Oops just read the ebuild submission doco's and have added bug 8114 as 
specified in http://www.gentoo.org/doc/ebuild-submit.html 
Comment 5 SpanKY gentoo-dev 2002-09-20 21:56:49 UTC
*** Bug 8114 has been marked as a duplicate of this bug. ***
Comment 6 Arcady Genkin (RETIRED) gentoo-dev 2002-09-23 01:39:56 UTC
Okay, I applied the patch to xine-d5d and created a similar patch for xine-d4d.
 Both plugins seem to work with xine-0.9.13 now.  Please test both packages. 
Thanks a lot for the report!