Skip to content

Commit

Permalink
cmake: make libxi and libxtst a requirement with autotype enabled
Browse files Browse the repository at this point in the history
  • Loading branch information
schlimmchen authored and droidmonkey committed Jun 19, 2022
1 parent 63a5e47 commit 861fe2e
Showing 1 changed file with 3 additions and 15 deletions.
18 changes: 3 additions & 15 deletions src/autotype/CMakeLists.txt
@@ -1,26 +1,14 @@
if(WITH_XC_AUTOTYPE)
if(UNIX AND NOT APPLE AND NOT HAIKU)
find_package(X11)
find_package(Qt5X11Extras 5.2)
find_package(X11 REQUIRED COMPONENTS Xi XTest)
find_package(Qt5X11Extras 5.2 REQUIRED)
if(PRINT_SUMMARY)
add_feature_info(libXi X11_Xi_FOUND "The X11 Xi Protocol library is required for auto-type")
add_feature_info(libXtst X11_XTest_FOUND "The X11 XTEST Protocol library is required for auto-type")
add_feature_info(Qt5X11Extras Qt5X11Extras_FOUND "The Qt5X11Extras library is required for auto-type")
endif()

# an error will be emitted by find_package above if X11 (or Qt5X11Extras) dev files were not found.
# X11 possibly is installed and found while X11_Xi and/or X11_XTest are not, so we emit a warning.
if(NOT X11_Xi_FOUND)
message(WARNING "The X11 Xi Protocol library is required for auto-type (try libxi-dev package)")
endif()

if(NOT X11_XTest_FOUND)
message(WARNING "The X11 XTEST Protocol library is required for auto-type (try libxtst-dev package)")
endif()

if(X11_FOUND AND X11_Xi_FOUND AND X11_XTest_FOUND AND Qt5X11Extras_FOUND)
add_subdirectory(xcb)
endif()
add_subdirectory(xcb)
elseif(APPLE)
add_subdirectory(mac)
elseif(WIN32)
Expand Down

1 comment on commit 861fe2e

@h1z1
Copy link

@h1z1 h1z1 commented on 861fe2e Jun 27, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious why would Qt 5.2 be a hard requirement?

Please sign in to comment.