[Kde-pim] Re: Review Request: Patch to build only kdecore

[Christophe Giboudeaux]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100697/#review1548
-----------------------------------------------------------



CMakeLists.txt
<http://git.reviewboard.kde.org/r/100697/#comment1320>

    Please find a better name for kdelibs. The module is not supposed to know what kimap is. (eg: KDE4_ONLY_KDECORE)
    
    



kdecore/localization/kcatalog.cpp
<http://git.reviewboard.kde.org/r/100697/#comment1321>

    where is this ifdef'd ?
    


- Christophe


On Feb. 21, 2011, 8:42 a.m., Gregory Schlomoff wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100697/
> -----------------------------------------------------------
> 
> (Updated Feb. 21, 2011, 8:42 a.m.)
> 
> 
> Review request for kdelibs and KDEPIM-Libraries.
> 
> 
> Summary
> -------
> 
> Hello,
> 
> As some of you may know, we are using KIMAP and KMime as stand-alone libraries for BetterInbox, our cross-platform email client.
> 
> Since our primary targets are Windows and Mac OS X platforms, it would be quite costly to try to build and ship the whole kdelibs + kdepimlibs + dependencies in order to use just kimap and kmime. So we made a few patches to KDE's build scripts that enable to build only kdecore + kimap + kmime. The only third-party dependencies on Windows are kdewin, boost, cyrus-sasl, and zlib.
> 
> Basically, we just declare a new build option called KIMAP_STANDALONE, that will skip the compilation of everything that is not needed. We are sharing the patches to kdelibs in this review request. See https://git.reviewboard.kde.org/r/100698/ for the patches to kdepimlibs.
> 
> Please note that the goal here is *NOT* to ask for the inclusion of those patches into KDE's source code, but rather to show the community how we did it, and maybe to foster some discussion around the idea of modularizing kdelibs and kdepimlibs a bit more, so that this kind of hack wouldn't be needed.
> 
> Any ideas / comments appreciated.
> 
> Cheers,
> 
> Greg Schlomoff
> Founder, BetterInbox
> 
> 
> Diffs
> -----
> 
>   kdecore/localization/kcatalog.cpp 1f01688 
>   kdecore/CMakeLists.txt f78c115 
>   cmake/modules/FindKDE4Internal.cmake d4e5780 
>   CMakeLists.txt 87631d1 
>   kdecore/tests/CMakeLists.txt 9703c7d 
> 
> Diff: http://git.reviewboard.kde.org/r/100697/diff
> 
> 
> Testing
> -------
> 
> Works flawlessly on Windows, Mac OS, and Ubuntu
> 
> 
> Thanks,
> 
> Gregory
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/