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

List:       kde-devel
Subject:    Re: [patch] New KEdit ready, please review
From:       Mickael Marchand <m.marcha () alemtid ! asso ! fr>
Date:       2001-11-26 22:54:44
[Download RAW message or body]

Hi,

i have tried that, adding :
#include <kparts/componentfactory.h>
#include <kregexpeditorinterface.h>

and 
regexpEditor = KParts::ComponentFactory::createInstanceFromQuery<QDialog>("KRegExpEditor/KRegExpEditor", \
QString::null, this ); in kedfind.cpp

and when compiling kdeui :
c++ -ansi -D_XOPEN_SOURCE=500 -D_BSD_SOURCE -Wbad-function-cast -Wcast-align -Wundef \
                -Wconversion -fno-builtin -Wnon-virtual-dtor
-Wno-long-long -g3 -Wall -pedantic -W -Wpointer-arith -Wmissing-prototypes \
-Wwrite-strings -fno-exceptions -fno-check-new -ftemplate-depth-99 \
-DQT_NO_TRANSLATION -DQT_CLEAN_NAMESPACE -DQT_NO_ASCII_CAST -DQT_NO_COMPAT -o \
.libs/libkdeui.la.closure .libs/libkdeui_la_closure.o .libs/libkdeui_la.all_cpp.o  \
-lpthread ../kdecore/.libs/libkdecore.so -L/opt/qt/lib -L/usr/X11R6/lib \
/kde/kdelibs/dcop/.libs/libDCOP.so -lpthread -L/opt/kde3/lib \
-L/usr/lib/gcc-lib/i386-linux/2.95.4 -ldl ../kdefx/.libs/libkdefx.so -lpthread \
-lpthread -lqt-mt -lpng -lz /usr/lib/libjpeg.so -lXext -lX11 -lresolv -lSM -lICE \
                -lstdc++ -lm -lc -lgcc -Wl,--rpath -Wl,/opt/kde3/lib 
.libs/libkdeui_la.all_cpp.o: In function \
`QValueListNode<KSharedPtr<KService>>::QValueListNode(KSharedPtr<KService> const &)': \
/kde/kdelibs/kdeui/../kdecore/ksharedptr.h(.QDialog * \
KParts::ComponentFactory::gnu.linkonce.t.createInstanceFromQuery<QDialog>(QString \
const &, QString const &, QObject *, char const *, QStringList const &, int *)+0x25): \
                undefined reference to `KTrader::self(void)'
/kde/kdelibs/kdeui/../kdecore/ksharedptr.h(.QDialog \
*KParts::ComponentFactory::gnu.linkonce.t.createInstanceFromService<QDialog>(KSharedPtr<KService>const \
&, QObject *, char const *, QStringList const &, int *)+0x38): undefined reference to \
`KService::library(void) const'

this looks like a missing lib :/

any idea ?

cheers,
Mik

On Mon, Nov 26, 2001 at 02:26:01PM +0100, Simon Hausmann wrote:
> On Mon, Nov 26, 2001 at 10:54:40AM +0100, David Faure wrote:
> > > It doesn't work for the texteditor stuff though because what's contained in 
> > > interfaces/ktexteditor are no real interfaces but base classes that 
> > > require linkage against a libktexteditor, which in turn links against 
> > > kparts. (that sucks, if you ask me :)
> > 
> > Ah, took me 2 times to understand this. Ok. Well, isn't that like
> > BrowserExtension, i.e. we have the problem of the moc-generated files,
> > whose symbols shouldn't be duplicated ?
> 
> The difference to BrowserExtension I see is that in the texteditor
> situation I see no real point in there being base classes instead of
> pure abstract interfaces (with BE it's needed actually because of
> the signals) . If it's for the duplicated API for the dcop interfaces 
> then I think this belongs into a separate support library apps can 
> optionally use. But implementing and using the interfaces IMHO shouldn't 
> _require_ linkage against a libktexteditor and therefore libkparts 
> (I think that's a design flaw, and in fact it prevents Mickael from
> using it in kdeui) .
> 
> > There isn't such a problem in the regexpeditor interface, right ?
> 
> Yep, no problem with the regxpeditor.
> 
> 
> Simon
> 
> > > Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

-- 
Mickael Marchand
Elève-ingénieur / Engineer student
Ecole des Mines de Douai - France
 
> > Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


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