review FindENCHANT.cmake

Yury G. Kudryashov urkud.urkud at gmail.com
Fri Sep 16 18:47:52 UTC 2011


Alexander Neundorf wrote:

> Hi,
> 
> same as with many others:
> 
> * unnecessary shortcut at the top
> * unnecessary if() around the pkgconfig stuff
> 
> Otherwise, it looks fine I'd say.
enchant installs its headers to $prefix/include/enchant by default. Thus the 
current code will not find enchant unless pkg-config information is 
available. Another question: should we force users to add 
$enchant_prefix/include/enchant to include_dirs(), or find_path( ... 
enchant/enchant++.h ...), and leave the decision whether to include 
$enchant_prefix/include or $enchant_prefix/include/enchant to the user?
-- 
Yury G. Kudryashov,
mailto: urkud at mccme.ru



More information about the Kde-buildsystem mailing list