By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1885 - kaffeine FTBFS - libcdio 0.90-2 drops <cdio/cdda.h>
Summary: kaffeine FTBFS - libcdio 0.90-2 drops <cdio/cdda.h>
Status: RESOLVED DUPLICATE of bug 1853
Alias: None
Product: TDE
Classification: Unclassified
Component: non-core programs (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: All Linux
: P5 critical
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2014-01-28 23:15 CST by David C. Rankin
Modified: 2014-01-29 00:50 CST (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Fix for libcdio-paranoia header move from cdio/ to cdio/paranoia (1.23 KB, patch)
2014-01-29 00:47 CST, David C. Rankin 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David C. Rankin 2014-01-28 23:15:50 CST 
The problem:

  cdio-paranoia is removed from libcdio > 0.83. A new package 'libcdio-paranoia' provides all old cdio-paranoia headers, BUT the header file location has moved from:

  ../cdio

to

  ../cdio/paranoia

  I successfully built kaffeine on Arch with libcdio & libcdio-paranoia by softlinking the needed paranoia headers to ../cdio:

cdda.h -> paranoia/cdda.h
paranoia.h -> paranoia/paranoia.h

  Obviously not a solution. How do we patch kaffeine to properly handle this change? kaffeine/src/input/disc/paranoia.h already includes:

#include <cdio/cdda.h>
#include <cdio/paranoia.h>

  The only other files that need fixing are:

configure.in:303:KDE_CHECK_HEADER([cdio/cdda.h], [with_cdparanoia=yes],
[with_cdparanoia=no])
kaffeine/configure.in.in:223:KDE_CHECK_HEADER([cdio/cdda.h],
[with_cdparanoia=yes], [with_cdparanoia=no])

  But how to do this with autofoo or preprocessor checks so that it will work with both libcdio <= 0.83 and those systems with libcdio > 0.83 that will also need libcdio-paranoia?

  I have elevated this to critical and not blocker because this is an application and not core, but still critical because this package will not build on current library releases and the fix should be simple.
Comment 1 Darrell 2014-01-29 00:11:00 CST 
Is this a duplicate of bug 1853?
Comment 2 David C. Rankin 2014-01-29 00:47:29 CST 
Created attachment 1904 [details]
Fix for libcdio-paranoia header move from cdio/ to cdio/paranoia

SOLVED FOR libcdio >= 0.9 - need automake conditional or preprocessor check to allow fix to work with both libcdio <= 0.83 and >= 0.9:

  Here is a patch that solves the problem for libcdio >= 0.9 where paranioa headers were removed from upstream libcdio and moved to a separate package libcdio-paranoia where they are installed in cdio/paranoia instead of cdio/
Comment 3 David C. Rankin 2014-01-29 00:50:43 CST 
(In reply to comment #1)
> Is this a duplicate of bug 1853?

Damn I knew the information sounded awful familiar. I even searched the database twice!! (but with k9copy as a search term --- oops) Closed :)

*** This bug has been marked as a duplicate of bug 1853 ***