@dmnks commented on this pull request.
> @@ -194,11 +194,14 @@ endif()
if (WITH_IMAEVM)
list(APPEND REQFUNCS lsetxattr)
- find_path(IMA_INCLUDE_DIR NAMES imaevm.h)
- find_library(IMA_LIBRARY NAMES imaevm)
- if (NOT (IMA_INCLUDE_DIR AND IMA_LIBRARY))
- message(FATAL_ERROR "imaevm enabled but library not present")
- endif()
+ find_path(IMA_INCLUDE_DIR NAMES imaevm.h REQUIRED)
+ find_library(IMA_LIBRARY NAMES imaevm REQUIRED)
REQUIRED was only added in cmake 3.18, though. But without it, one has to use
these awkward checks and custom error messages...
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2472#pullrequestreview-1374319415
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/2472/review/1374319...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint