Re: patch: c/cxxflags for static libraries

2006-07-19 Thread Thiago Macieira
Matt Broadstone wrote:
Well its not so much that I want a convenience lib, as I want a
logical breakdown of the parts within a whole. For me it makes a lot
more sense to build khtml in its pieces: dom, ecma, java, html etc.
and then link them together in the whole. Furthermore, when I've been
using my patch to build khtml, when I make simple changes inside
ecma/debugger I no longer have to build the entirety of khtml. This
may be a problem with cmake, as this should not necesarily be the case
considering I haven't touched code that affects all of khtml, but
that's what happens. If we can find a way around this, ie. change the
CMakeLists.txt for khtml in such a way that simple changes in each
module do not affect the entirety of khtml, then that's a decent
solution. But for now, on my box, doing it this way (convenience,
modular, whatever you want to call it :) ) works better for me, and I
simply thought it might work better for others as well.

I don't see how your solution can work. If you modify one file, you must 
compile it then link libkhtml. Isn't that what cmake is doing?

How would introducing an intermediary step (create a .a) improve things?

-- 
  Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358


pgpycG8KR10x9.pgp
Description: PGP signature
___
Kde-buildsystem mailing list
Kde-buildsystem@kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem


small patch imlib and kuickshow

2006-07-19 Thread Marcus Hufgard \(Kalkwerk Hufgard GmbH\)
Hi!

I have a small cmake-patch for building kuickshow. Please have a look at
it and commit this too trunk.

Thanks

By

Marcus




Index: kdegraphics/cmake/modules/FindIMLIB.cmake
===
--- kdegraphics/cmake/modules/FindIMLIB.cmake	(Revision 564004)
+++ kdegraphics/cmake/modules/FindIMLIB.cmake	(Arbeitskopie)
@@ -20,7 +20,7 @@
   # in the FIND_PATH() and FIND_LIBRARY() calls
   INCLUDE(UsePkgConfig)
   
-  PKGCONFIG(libagg _IMLIBIncDir _IMLIBLinkDir _IMLIBLinkFlags _IMLIBCflags)
+  PKGCONFIG(imlib _IMLIBIncDir _IMLIBLinkDir _IMLIBLinkFlags _IMLIBCflags)
   
   set(IMLIB_DEFINITIONS ${_IMLIBCflags})
   
@@ -30,17 +30,17 @@
 /usr/local/include
   )
   
-  FIND_LIBRARY(IMLIB_LIBRARIES NAMES Imlib
+  FIND_LIBRARY(IMLIB_LIBRARIES NAMES Imlib libImlib
 PATHS
 ${_IMLIBLinkDir}
 /usr/lib
 /usr/local/lib
   )
-  
+
   if (IMLIB_INCLUDE_DIR AND IMLIB_LIBRARIES)
- SET(imlib_found true)
+ SET(IMLIB_FOUND true)
   endif (IMLIB_INCLUDE_DIR AND IMLIB_LIBRARIES)
-  
+ 
   if (IMLIB_FOUND)
 set(CACHED_IMLIB YES)
 if (NOT IMLIB_FIND_QUIETLY)
@@ -48,6 +48,9 @@
 endif (NOT IMLIB_FIND_QUIETLY)
   else (IMLIB_FOUND)
 set(CACHED_IMLIB NO)
+if (NOT IMLIB_FIND_QUIETLY)
+  message(STATUS didn't find IMLIB)
+endif (NOT IMLIB_FIND_QUIETLY)
 if (IMLIB_FIND_REQUIRED)
   message(FATAL_ERROR Could NOT find IMLIB)
 endif (IMLIB_FIND_REQUIRED)
Index: kdegraphics/CMakeLists.txt
===
--- kdegraphics/CMakeLists.txt	(Revision 564004)
+++ kdegraphics/CMakeLists.txt	(Arbeitskopie)
@@ -12,10 +12,12 @@
 macro_optional_find_package(Gphoto2)
 macro_optional_find_package(Sane)
 macro_optional_find_package(OpenGL)
+macro_optional_find_package(IMLIB)
 
 MACRO_LOG_FEATURE(GPHOTO2_FOUND libgphoto2 Need to build kamera project http://heanet.dl.sourceforge.net/sourceforge/gphoto/; FALSE  )
 MACRO_LOG_FEATURE(SANE_FOUND libsane Sane lib is needed to build kooka project http://www.sane-project.org/; FALSE  )
-MACRO_LOG_FEATURE(OPENGL_FOUND libopengl OpenGl lib is need to build kpovmodeler project http://www.X.org; FALSE  )
+MACRO_LOG_FEATURE(OPENGL_FOUND libopengl OpenGl lib is need to build kpovmodeler project http://www.X.org/; FALSE  )
+MACRO_LOG_FEATURE(IMLIB_FOUND imlib imlib is need to build kuickshow project http://www.gnome.org/; FALSE  )
 
 # are these two really required ?
 find_package(ZLIB REQUIRED)
@@ -66,7 +68,8 @@
 add_subdirectory(kruler)
 add_subdirectory(ksnapshot)
 if (IMLIB_FOUND)
-   add_subdirectory(kuickshow)
+   MESSAGE(STATUS port kuickshow!)
+   #add_subdirectory(kuickshow)
 endif (IMLIB_FOUND)
 add_subdirectory(kviewshell)
 
___
Kde-buildsystem mailing list
Kde-buildsystem@kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem


Re: small patch imlib and kuickshow

2006-07-19 Thread Laurent Montel
On Wednesday 19 July 2006 08:36, Marcus Hufgard (Kalkwerk Hufgard GmbH) wrote:
 Hi!

 I have a small cmake-patch for building kuickshow. Please have a look at
 it and commit this too trunk.

Apply thanks.
Regards

 Thanks

 By

 Marcus
___
Kde-buildsystem mailing list
Kde-buildsystem@kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem


Re: Convenience Libraries (was RFC: KHTML modular build)

2006-07-19 Thread Dirk Mueller
On Monday 17 July 2006 20:09, Matt Broadstone wrote:

 Hi, I submitted a patch to kfm-devel to convert khtml to use
 libtool-esque convenience libraries yesterday, and wanted to move the
 discussion over here as it seems more appropriate.

What is the reason for that?


Dirk
___
Kde-buildsystem mailing list
Kde-buildsystem@kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem


Re: patch: c/cxxflags for static libraries

2006-07-19 Thread Matt Broadstone
On 7/19/06, Thiago Macieira [EMAIL PROTECTED] wrote:
 Matt Broadstone wrote:
 Well its not so much that I want a convenience lib, as I want a
 logical breakdown of the parts within a whole. For me it makes a lot
 more sense to build khtml in its pieces: dom, ecma, java, html etc.
 and then link them together in the whole. Furthermore, when I've been
 using my patch to build khtml, when I make simple changes inside
 ecma/debugger I no longer have to build the entirety of khtml. This
 may be a problem with cmake, as this should not necesarily be the case
 considering I haven't touched code that affects all of khtml, but
 that's what happens. If we can find a way around this, ie. change the
 CMakeLists.txt for khtml in such a way that simple changes in each
 module do not affect the entirety of khtml, then that's a decent
 solution. But for now, on my box, doing it this way (convenience,
 modular, whatever you want to call it :) ) works better for me, and I
 simply thought it might work better for others as well.

 I don't see how your solution can work. If you modify one file, you must
 compile it then link libkhtml. Isn't that what cmake is doing?

 How would introducing an intermediary step (create a .a) improve things?

What I am referring to is that when I'm working on khtml (in this
example) and make a change to say dom/, in my build dir I can cd to
kdelibs/build/khtml/dom and build from that directory instead of the
khtml directory and it will only rebuild the dom lib and then link
things accordingly. Perhaps it should be the case when I make the
change and build from khtml/, but it doesn't work that way - when I
build from khtml in that example it rebuilds all of khtml. That's all
I'm trying to get around...

Matt Broadstone


 --
  Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
   PGP/GPG: 0x6EF45358; fingerprint:
   E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358


 ___
 Kde-buildsystem mailing list
 Kde-buildsystem@kde.org
 https://mail.kde.org/mailman/listinfo/kde-buildsystem




___
Kde-buildsystem mailing list
Kde-buildsystem@kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem


Re: patch: c/cxxflags for static libraries

2006-07-19 Thread Leo Savernik
Am Mittwoch, 19. Juli 2006 17:06 schrieb Matt Broadstone:
 What I am referring to is that when I'm working on khtml (in this
 example) and make a change to say dom/, in my build dir I can cd to
 kdelibs/build/khtml/dom and build from that directory instead of the
 khtml directory and it will only rebuild the dom lib 

Under autotools, you'd have got a libdom.a convenience lib by that time. So no 
other files are recompiled than those this lib depends onto. So far so good.

 and then link 
 things accordingly. 

And now it gets interesting.
Did you use automake in this step formerly (i. e. make all in khtml/)? If 
yes, it'll do dependency checking and *will* recompile all affected files, 
will rebuild all changed convenience libs and only then will relink 
libkhtml.so.

So unless you did fancy stuff the default way to build things did never ever 
relieve one from rebuilding necessary dependencies under autotools.

 Perhaps it should be the case when I make the 
 change and build from khtml/, but it doesn't work that way - when I
 build from khtml in that example it rebuilds all of khtml. That's all
 I'm trying to get around...

The general problem in khtml is that it's tightly coupled, i. e. nearly every 
header includes nearly every other header, transitively. So what may look 
like a total recompile could simply be a large set of files affected by 
transitive includes.

OTOH, if cmake is really rebuilding *everything*, it looks like more a bug in 
the buildsystem.

mfg
Leo
___
Kde-buildsystem mailing list
Kde-buildsystem@kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem