@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)
Which makes me think if we shouldn't just bump our minimal cmake version?
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2472#discussion_r1159460301
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/2472/review/1374328...@github.com>_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint
