----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/142/#review292 -----------------------------------------------------------
Ship it! looks good. please commit. - Matt On 2009-02-21 01:14:16, gja wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/142/ > ----------------------------------------------------------- > > (Updated 2009-02-21 01:14:16) > > > Review request for Kopete, Matt Rogers and gja. > > > Summary > ------- > > Well, this is more of a patch for developers. It sets the directory which is > used for includes as /kdenetwork/kopete ratherthan /kdenetwork. This has no > effects to users or even packagers. My use case is that I have a > /src/kdenetwork/kopete (From trunk) and a /src/branch/kopete-dev-4.2. > > Kopete can now be built outside of kdenetwork just by adding the following > lines to Kopete's main CMakeLists.txt (all of these lines may not be needed): > find_package(KDE4 REQUIRED) > include (KDE4Defaults) > include (MacroLibrary) > > include(CheckIncludeFile) > include(CheckIncludeFiles) > include(CheckSymbolExists) > include(CheckFunctionExists) > include(CheckLibraryExists) > include(CheckPrototypeExists) > include(CheckTypeSize) > include(MacroBoolTo01) > include(MacroOptionalAddSubdirectory) > > find_package(KdepimLibs REQUIRED) > > add_definitions (${QT_DEFINITIONS} ${KDE4_DEFINITIONS}) > include_directories (${KDE4_INCLUDES} ${KDEPIMLIBS_INCLUDE_DIR}) > > Of course, this is only useful if you plan on having multiple trees of kopete > on your system which share a single environment. This way, I can have > multiple trees of kopete, and need not worry about a separate kdenetwork tree > for each of them. Yay! > > Respond on kopete mailing list if i can commit this > > > Diffs > ----- > > /dev/null 928222 > /dev/null 928222 > /dev/null 928222 > /dev/null 928222 > /dev/null 928222 > /dev/null 928222 > /dev/null 928222 > /dev/null 928222 > /dev/null 928222 > /trunk/KDE/kdenetwork/CMakeLists.txt 928222 > /trunk/KDE/kdenetwork/cmake/modules/FindDecibel.cmake 928222 > /trunk/KDE/kdenetwork/cmake/modules/FindIDN.cmake 928222 > /trunk/KDE/kdenetwork/cmake/modules/FindLibMeanwhile.cmake 928222 > /trunk/KDE/kdenetwork/cmake/modules/FindLibOTR.cmake 928222 > /trunk/KDE/kdenetwork/cmake/modules/FindLibgadu.cmake 928222 > /trunk/KDE/kdenetwork/cmake/modules/FindLibmsn.cmake 928222 > /trunk/KDE/kdenetwork/cmake/modules/FindLiboRTP.cmake 928222 > /trunk/KDE/kdenetwork/cmake/modules/FindSpeex.cmake 928222 > /trunk/KDE/kdenetwork/cmake/modules/ortpversion.c 928222 > /trunk/KDE/kdenetwork/kopete/CMakeLists.txt 928222 > /trunk/KDE/kdenetwork/kopete/kopete/CMakeLists.txt 928222 > /trunk/KDE/kdenetwork/kopete/kopete/config/accounts/CMakeLists.txt 928222 > /trunk/KDE/kdenetwork/kopete/kopete/config/chatwindow/CMakeLists.txt 928222 > /trunk/KDE/kdenetwork/kopete/kopete/identity/CMakeLists.txt 928222 > /trunk/KDE/kdenetwork/kopete/protocols/CMakeLists.txt 928222 > /trunk/KDE/kdenetwork/kopete/protocols/groupwise/CMakeLists.txt 928222 > > /trunk/KDE/kdenetwork/kopete/protocols/groupwise/libgroupwise/CMakeLists.txt > 928222 > > /trunk/KDE/kdenetwork/kopete/protocols/groupwise/libgroupwise/tasks/tests/CMakeLists.txt > 928222 > > /trunk/KDE/kdenetwork/kopete/protocols/groupwise/libgroupwise/tests/CMakeLists.txt > 928222 > /trunk/KDE/kdenetwork/kopete/protocols/oscar/liboscar/tests/CMakeLists.txt > 928222 > /trunk/KDE/kdenetwork/kopete/protocols/yahoo/libkyahoo/tests/CMakeLists.txt > 928222 > > Diff: http://reviewboard.kde.org/r/142/diff > > > Testing > ------- > > It builds here :) > > I set KOPETE_FOO_DIR to ${CMAKE_CURRENT_FOO_DIR} in the beginning, which is > guaranteed to work unless CMake Borks someday in the future. > > All paths are relative. > > > Thanks, > > gja > > _______________________________________________ kopete-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kopete-devel
