Standalone qt-dbus in kdesupport
Paulo Jorge Guedes
Paulo.Guedes at artelecom.pt
Thu Jun 8 19:45:27 BST 2006
> -----Original Message-----
> From: Paulo Jorge Guedes [mailto:Paulo.Guedes at artelecom.pt]
> Sent: quinta-feira, 8 de Junho de 2006 19:31
> To: kde-core-devel at kde.org
> Subject: RE: Standalone qt-dbus in kdesupport
>
> > > > > >So, what do you think of
> > > > > >
> > > > > >--- CMakeLists.txt (revision 549165)
> > > > > >+++ CMakeLists.txt (working copy)
> > > > > >@@ -1 +0,0 @@
> > > > > >-ADD_SUBDIRECTORY( qt-dbus )
> > > > >
> > > > > Huh? Why do you want to disable qt-dbus builds?
> > > >
> > > > No he wants people to run cmake from within qt-dbus instead of
> doing
> > > it
> > > > for the whole of kdesupport in one go.
> > > >
> > > > As long as kdesvn-build supports it, I don't mind too much
> > > > (right now I install kdesupport into its own prefix, which is a
> bit
> > > > strange
> > > > since it's a collection of unrelated things).
> > >
> > > Precisely.
> > > People can always do cmake ..\kdesupport\qt-dbus in some qt-dbus
> build
> > > directory but IMHO it would be better if they were forced to (just
> like
> > > people build kdewin32) because, like David said, it's a collection
> of
> > > unrelated things.
> >
> > But if both are possible in the current situation, then I don't see
> the
> > point in forcing
> > anything upon people...
>
> Then perhaps we should change FindQtDbus.cmake, because it won't find
> nothing on windows if one does make install on kdesupport.
Something like this:
===================================================================
--- FindQtDBus.cmake (revision 549359)
+++ FindQtDBus.cmake (working copy)
@@ -28,7 +28,7 @@
else (QDBUS_INCLUDE_DIRS AND QDBUS_LIBRARIES)
if (WIN32)
- find_file(QDBUS_DIR qt-dbus
+ find_file(QDBUS_DIR kdesupport
${_progFiles}
"C:/"
)
===================================================================
--- CMakeLists.txt (revision 549165)
+++ CMakeLists.txt (working copy)
@@ -1 +1,7 @@
+PROJECT(kdesupport)
+
+if(WIN32)
+ ADD_SUBDIRECTORY( kdewin32 )
+endif(WIN32)
+
ADD_SUBDIRECTORY( qt-dbus )
===================================================================
FindKDEWIN32.cmake should also be changed. Perhaps it could use the same
approach of FindQtDBus on Windows, and first try to find a KDEWIN32_DIR
(like the QDBUS_DIR).
This would make more sense. As it is right now, it's just not coherent.
If you find this sensitive, I can present a complete patch.
Paulo
More information about the kde-core-devel
mailing list