[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-linux
Subject: Re: [kde-linux] kde 3.3.1 can't be compiled here
From: James Richard Tyrer <tyrerj () acm ! org>
Date: 2004-10-30 21:47:54
Message-ID: 41840C0A.20200 () acm ! org
[Download RAW message or body]
Christian H. Kuhn wrote:
> & Jim Philips <jimmyc@speedfactory.net> [2004-10-30 17:16]:
>
>>http://lists.kde.org/?l=kde-devel&m=108947887823347&w=2
>
>
> It works. thx a lot.
The official fix is slightly simpler:
--- kdemultimedia/mpeglib/lib/input/cdromAccess_Linux.cpp 2004/04/19 \
01:12:55 1.7
+++ kdemultimedia/mpeglib/lib/input/cdromAccess_Linux.cpp 2004/09/08 \
14:41:44 1.8
@@ -10,12 +10,12 @@
*/
+#include <linux/types.h>
+#include <linux/cdrom.h>
+#include <sys/ioctl.h>
#include "cdromToc.h"
#include "cdromRawAccess.h"
-#include <asm/types.h>
-#include <linux/cdrom.h>
-#include <sys/ioctl.h>
#include <iostream>
So there was an error in the code that is obvious not that someone else fixed
it. :-)
But there is some question if this will work in all cases:
http://bugs.kde.org/show_bug.cgi?id=86188
There are bugs in some versions of the Linux headers.
So additional changes were made on HEAD.
--- kdemultimedia/mpeglib/lib/input/cdromAccess_Linux.cpp #1.8:1.9
at at -11,6 +11,13 at at
*/
+#ifdef __STRICT_ANSI__
+#undef __STRICT_ANSI__
+#define _ANSI_WAS_HERE_
+#endif
#include <linux/types.h>
#include <linux/cdrom.h>
+#ifdef _ANSI_WAS_HERE_
+#define __STRICT_ANSI__
+#endif
#include <sys/ioctl.h>
Still don't know exactly why it worked for me but not for you, but I suspect
that the problem in the Linux headers has been fixed with version >= 2.4.26
The other patch (that you used) should work fine on Linux but may not work on
other systems.
--
JRT
___________________________________________________
This message is from the kde-linux mailing list.
Account management: https://mail.kde.org/mailman/listinfo/kde-linux.
Archives: http://lists.kde.org/.
More info: http://www.kde.org/faq.html.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic