trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: February 2014

Re: [trinity-devel] kaffeine FTBFS - kaffeine/configure.in.in also needs patch for paranoia/cdda.h

From: "David C. Rankin" <drankinatty@...>
Date: Wed, 05 Feb 2014 10:41:56 -0600
On 02/05/2014 10:19 AM, David C. Rankin wrote:
> On 02/05/2014 10:00 AM, David C. Rankin wrote:
>> Should be:
>>
>> extern "C"
>> {
>> #ifdef HAVE_LIBCDIO_090
>> #include <cdio/paranoia/cdda.h>
> 
> I think this whole issue is that HAVE_LIBCDIO_090 is NOT seen in paranoia.h. If
> we can get that test to evaluate to 'true', I think kaffeine will build.
> 

That is indeed the case. I just built kaffeine with:

cd kaffeine

# fix kaffeine/kaffeine/configure.in.in
sed -i 's|\[cdio/cdda.h\]|[cdio/paranoia/cdda.h]|' kaffeine/configure.in.in

# set all paranoia.h 'cdio/' includes to 'cdia/paranoia/'
sed -i 's|<cdio/cdda.h>|<cdio/paranoia/cdda.h>|' \
  kaffeine/src/input /disc/paranoia.h
sed -i 's|<cdio/paranoia.h>|<cdio/paranoia/paranoia.h>|' \
  kaffeine/src/input /disc/paranoia.h

# fix missing '/' in kaffeine/kaffeine/src/input/disc/paranoia.h
sed -i 's|paranoiacdda|paranoia/cdda|' kaffeine/src/input/disc/paranoia.h

Kaffeine built fine.  So we have to:

(1) fix the missing '/' at line 36 in paranoia.h:

 33 extern "C"
 34 {
 35 #ifdef HAVE_LIBCDIO_090
 36 #include <cdio/paranoiacdda.h>
                           ^

(2) find a way to get HAVE_LIBCDIO_090 test true in paranoia.h

How do we fix #2?

-- 
David C. Rankin, J.D.,P.E.