[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: Re: kdebase --enable-final fix
From:       fenix () club-internet ! fr
Date:       2002-05-30 20:55:55
[Download RAW message or body]




----Message d'origine----
>De: David Faure <david@mandrakesoft.com>
>A: kde-devel@kde.org
>Sujet: Re: kdebase --enable-final fix
>Date: Thu, 30 May 2002 22:47:05 +0200
>
>On Friday 31 May 2002 04:12, fenix wrote:
>> Le jeu 30/05/2002 à 11:28, patrol@sinus.cz a écrit :
>> > Hi!
>> >   Please look at it and commit if no objections against it...
>> >                                 With regards, Pavel Troller
>> > ----
>> > 
>> 
>> Hi,
>>  can anyone apply this patch ... please 
>> 
>>  FeniX
>> 
>> > ? diff.out
>> > Index: main.cc
>> > ===================================================================
>> > RCS file: /home/kde/kdebase/kdesktop/main.cc,v
>> > retrieving revision 1.78
>> > diff -u -3 -p -r1.78 main.cc
>> > --- main.cc	7 Mar 2002 16:01:39 -0000	1.78
>> > +++ main.cc	30 May 2002 15:18:21 -0000
>> > @@ -17,6 +17,8 @@
>> >     Boston, MA 02111-1307, USA.
>> >  */
>> >  
>> > +#undef index
>> > +
>> >  #include <kuniqueapplication.h>
>> >  #include <klocale.h>
>> >  #include <dcopclient.h>
>
>I see no code using "index" in kdesktop.
>So I assume it's one of the kdelibs headers that does?
>In that case it would seem wiser to add the undef in that header,
>to prevent the problem from repeating itself somewhere else.

No ... if i remember its the X11 Xos.h header who define the "index" macro ;(

FeniX

>-- 
>David FAURE, david@mandrakesoft.com, faure@kde.org
>http://people.mandrakesoft.com/~david/
>Contributing to: http://www.konqueror.org/, http://www.koffice.org/
>KDE, Making The Future of Computing Available Today
>
>
> 
>>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
>
>


>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic