D8998: Add FindSeccomp to find-modules
Christophe Giboudeaux
noreply at phabricator.kde.org
Sun Nov 26 09:57:31 UTC 2017
cgiboudeaux added inline comments.
INLINE COMMENTS
> FindSeccomp.cmake:12
> +# The seccomp include directories
> +# ``Seccomp_LIBRARIES``
> +# The seccomp libraries for linking
so, what about naming your variables Seccomp_LIBRARIES and Seccomp_INCLUDE_DIRS in the file ?
> FindSeccomp.cmake:24
> +#=============================================================================
> +# Copyright (c) 2017 Martin Flöser <mgraesslin at kde.org>, David Kahles <david.kahles96 at gmail.com>
> +#
One author per line
> FindSeccomp.cmake:56
> +
> +find_path(Seccomp_INCLUDE_DIR
> + NAMES
Seccomp_INCLUDE_DIRS
> FindSeccomp.cmake:62
> +)
> +find_library(Seccomp_LIBRARY
> + NAMES
Seccomp_LIBRARIES
> FindSeccomp.cmake:74-75
> + REQUIRED_VARS
> + Seccomp_LIBRARY
> + Seccomp_INCLUDE_DIR
> + VERSION_VAR
same thing here
> FindSeccomp.cmake:83-85
> + IMPORTED_LOCATION "${Seccomp_LIBRARY}"
> + INTERFACE_COMPILE_OPTIONS "${Seccomp_DEFINITIONS}"
> + INTERFACE_INCLUDE_DIRECTORIES "${Seccomp_INCLUDE_DIR}"
and there
> FindSeccomp.cmake:89
> +
> +mark_as_advanced(Seccomp_LIBRARY Seccomp_INCLUDE_DIR)
> +
here as well
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D8998
To: davidk, graesslin
Cc: cgiboudeaux, #frameworks, #build_system
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20171126/1c7d29c8/attachment.html>
More information about the Kde-buildsystem
mailing list