>
> A few CMake tips:
>  * The if() block is not needed, as find_package_handle_standard_args is
>   responsible for setting KrossPython_FOUND if KROSSPYTHON_LIBRARY has
>   been found (pay attention to the capitalization).
>
>  * You should not need to pass additional PATHS to find_library(),
>   however you might need to pass "kde4" to PATH_SUFFIXES (in fact,
>   hardcoding "kde4" is not the best solution; if there's not
>   appropriate variable, you could ping us in the kde-buildsystem
>   mailing list).
>
>  * After that, you should check your CMAKE_PREFIX_PATH and see if
>   contains the directory which has krosspython.so.
>
> Now find_library is probably not finding krosspython because it does not
> start with the 'lib' prefix (ie it's not libkrosspython.so) -- try
> appending ".${CMAKE_SHARED_LIBRARY_SUFFIX}", as in:
>
>    find_library(FOO_LIBRARY
>                 NAMES "krosspython${CMAKE_SHARED_LIBRARY_SUFFIX}")
>

 I noticed that here you said ".${CMAKE_SHARED_LIBRARY_SUFFIX}" using a the
dot, and in the example you gave you don't use it. Anyway, I tried both and
it doesn't find the library using your recommendations (lib prefix,
capitalization). I also checked my CMAKE_PREFIX_PATH and it didn't have any
path so I added the corresponding one where krosspython.so is located and
the results were the same, CMake doesn't find the library.


It sounds like you want to do a build time check on a runtime
> requirement (else, I don't understand why want to do this)
>
> Please don't do build time checks for runtime things that actually
> changes the result.
>

Yes, this CMake code is to check whether or not the krosspython library is
installed on the system when  you execute the configure file before build.
We think it is necessary to check it because it allows the user to know
which functionalities you will have. As krosspython is used to be able to
run Python plugins via Kross, the user can decide whether or not to have
those features.

Btw, what do you mean by "runtime things that actually changes the result"?

Thanks,

-- 
Daniel E. Moctezuma
KMess developer team & Japanese translation
------
KMess
- the Live Messenger alternative for KDE -
web: www.kmess.org - irc: #kmess at Freenode
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

Reply via email to