[Kde-pim] Review Request 127228: [kleopatra] Drop duplicate check from root and make it optional

[Andreas Sturmlechner]

> On March 1, 2016, 6:22 a.m., Laurent Montel wrote:
> > CMakeLists.txt, line 224
> > <https://git.reviewboard.kde.org/r/127228/diff/1/?file=446050#file446050line224>
> >
> >     check is still necessary.

The exact same check is done in kleopatra/CMakeLists.txt (of course there it is fatal), so it's an opportunity for a small cleanup


> On March 1, 2016, 6:22 a.m., Laurent Montel wrote:
> > CMakeLists.txt, line 236
> > <https://git.reviewboard.kde.org/r/127228/diff/1/?file=446050#file446050line236>
> >
> >     kleopatra is necessary for kmail too why you want to disable build ?

Even the existing logic makes kleopatra conditional to Boost_TOPOLOGICAL_SORT_DIR and Gpemepp, so that's not the impression I got? But this is not about simply disabling kleopatra anyway - it can be built completely separate within its own directory, and if you do that, you want to be able to disable it for the rest of kdepim.


- Andreas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127228/#review92962
-----------------------------------------------------------


On Feb. 29, 2016, 10:03 p.m., Andreas Sturmlechner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127228/
> -----------------------------------------------------------
> 
> (Updated Feb. 29, 2016, 10:03 p.m.)
> 
> 
> Review request for KDEPIM, Andre Heinecke and Laurent Montel.
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> Same check exists in kleopatra/CMakeLists.txt already.
> 
> Going forward, ecm_optional_add_subdirectory provides a convenient handle to switch kleopatra on or off.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 63384efb9c495bd3f9f2b507ef747cac7844deaa 
> 
> Diff: https://git.reviewboard.kde.org/r/127228/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andreas Sturmlechner
> 
>

_______________________________________________
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/