Details
Details
- Reviewers
dvratil mlaurent - Group Reviewers
FreeBSD - Commits
- R94:ca2fc3a8df06: Use ecm's FindInotify.cmake
Diff Detail
Diff Detail
- Repository
- R94 PIM: Message Library
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage
messageviewer/src/CMakeLists.txt | ||
---|---|---|
28 | Just use ${Inotify_FOUND} here directly instead of SYS_INOTIFY_H_FOUND and you can remove the line above. |
messageviewer/src/CMakeLists.txt | ||
---|---|---|
31 | I think this is wrong. The macro_bool_to_01 evaluates the first argument and then sets the remaining arguments to 0 or 1 based on it, so it should be: macro_bool_to_01(Inotify_FOUND HAVE_SYS_INOTIFY_H) |
Comment Actions
Oh sorry.
Btw, why isn't it using #cmakedefine01? Isn't marcro_bool_to_01 deprecated?
Comment Actions
macro_bool_to_01 is indeed deprecated, and config-messageviewer.h contains HAVE_SYS_INOTIFY_H so maybe we can replace macro_bool_to_01 just by set(HAVE_SYS_INOTIFY_H ${Inotify_FOUND}).
Feel free to adjust it if you want to, but the patch is good to go the way it is now.