[cmake-developers] include(...) or find_package(...) within another Findpackage module

2014-09-09 Thread Richard Shaw
I'm working on trying to modernize the FindFLTK module but also stuck with making sure I don't break the established behavior. The problems with the module are extensive so I'm breaking them up into individual emails where possible. The part I'm questioning in this email is: Should this be

Re: [cmake-developers] include(...) or find_package(...) within another Findpackage module

2014-09-09 Thread Brad King
On 09/09/2014 02:21 PM, Richard Shaw wrote: Should this be include(FindX11) as it shows or find_package(X11)? if(UNIX) include(FindX11) find_library(FLTK_MATH_LIBRARY m) set(FLTK_PLATFORM_DEPENDENT_LIBS ${X11_LIBRARIES} ${FLTK_MATH_LIBRARY}) endif() It should be find_package(X11).