Alexander Neundorf wrote: > Hi, > > I think this one looks good as it is. > If somebody wants to, it could be changed to all-lowercase and empty > endif() statements.
Are you talking about attic/modules/FindFlex.cmake (didn't found FindFLEX.cmake)? If yes, I see two problems: 1. FLEX_LIBRARY and FLEX_INCLUDE_DIR are not documented; 2. It is not possible to specify whether one needs both executable and library+headers, or just executable, or just library. What about find_package(Flex COMPONENTS EXECUTABLE LIBRARY)? About the second point, I never used flex, so probably one never need only one of these components. In this case, just ignore this point. -- Yury G. Kudryashov, mailto: [email protected] _______________________________________________ Kde-buildsystem mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-buildsystem
