[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-commits
Subject: branches/KDE/3.5/kdemultimedia
From: Adriaan de Groot <groot () kde ! org>
Date: 2006-01-15 23:52:42
Message-ID: 1137369162.306933.28337.nullmailer () svn ! kde ! org
[Download RAW message or body]
SVN commit 498711 by adridg:
Take all the weird hacks that were done specifically for FreeBSD
and get rid of them. The upstream cdparanoia has been fixed now,
which gives us a more straightforward interface.
CCMAIL: kde@freebsd.org
M +22 -0 configure.in.in
M +1 -2 kioslave/audiocd/audiocd.cpp
--- branches/KDE/3.5/kdemultimedia/configure.in.in #498710:498711
@@ -251,6 +251,28 @@
KDE_CHECK_HEADER(cdda_interface.h,
[:], [have_cdparanoia=no])
+ # Older versions of FreeBSD's cdparanoia lack cdrom_drive.cdda_device_name
+ AC_MSG_CHECKING([for cdrom_drive.cdda_device_name in cdda_interface.h])
+ kde_save_CPPFLAGS="$CPPFLAGS"
+ CPPFLAGS="$CPPFLAGS $all_includes"
+ AC_LANG_SAVE
+ AC_LANG_C
+ AC_TRY_COMPILE([
+ #include <cdda_interface.h>
+ ],
+ [
+ struct cdrom_drive device;
+ device.cdda_device_name = 0;
+ ],
+ [
+ AC_MSG_RESULT([yes])
+ ], [
+ AC_MSG_RESULT([no])
+ have_cdparanoia=no
+ ])
+ AC_LANG_RESTORE
+ CPPFLAGS=$kde_save_CPPFLAGS
+
KDE_CHECK_LIB(cdda_paranoia, paranoia_init,
[:], [have_cdparanoia=no], [-lcdda_interface -lm])
--- branches/KDE/3.5/kdemultimedia/kioslave/audiocd/audiocd.cpp #498710:498711
@@ -215,9 +215,8 @@
// Update our knowledge of the disc
// TODO which one is right?
// qDebug("\"%s\" \"%s\"", drive->cdda_device_name, drive->ioctl_device_name);
-#if defined(Q_OS_LINUX)
d->cd.setDevice(drive->cdda_device_name, 50, false);
-#elif defined(Q_OS_FREEBSD)
+#if 0
// FreeBSD's cdparanoia as of january 5th 2006 has rather broken
// support for non-SCSI devices. Although it finds ATA cdroms just
// fine, there is no straightforward way to discover the device
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic