On 06/08/2014 09:12 AM, o.k...@gmx.de wrote:
> I deleted the additional "include/" in FindLua51.cmake and it worked.

I think that is the correct fix.  I do not know why they were there
in the first place.  They appear in FindLua50, FindLua51, and the
version-agnostic FindLua module.  Eike, did you just preserve that
idiom when creating the latter module or did you discover the reason
the include/ is needed in PATH_SUFFIXES?

Also I would appreciate a generalized path that covers all the
modules.  Explain in the commit message why the behavior of the
find_path command's PATH_SUFFIXES option makes the include/
portion of the path unnecessary.  (Actually I think the trailing
"include" can be dropped too).

Thanks,
-Brad

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers

Reply via email to