Commit f013a85b authored by Harald Sitter's avatar Harald Sitter
Browse files

properly show xfreerdp runtime dependency in cmake summary

in absence of a proper finder we'll have a fake package find which we
force found depending on the find_program call.

this results in the runtime dep being reported as such in the cmake
feature summary

Test Plan:
`rm -rf *; cmake ..`

- executable found => reported found in summary
- executable !found => reported !found

Reviewers: #kde_applications, aacid

Reviewed By: #kde_applications, aacid

Subscribers: aacid, whiting

Differential Revision:
parent c3ba7c0e
......@@ -83,14 +83,22 @@ set_package_properties("libssh" PROPERTIES
# Needs porting
find_program(FREERDP_EXECUTABLE xfreerdp)
set(FREERDP_FOUND true) # mark the package found
set_package_properties("freerdp" PROPERTIES
# dud to allow the dep showing up in feature summary.
# note that this must be after _FOUND is set so it doesn't incorrectly
# set the metadata to not-found.
find_package(freerdp QUIET)
set_package_properties(freerdp PROPERTIES
DESCRIPTION "A free Remote Desktop Protocol (RDP) Implementation"
URL ""
PURPOSE "Needed for RDP support in KRDC (at runtime)"
PURPOSE "The xfreerdp binary is needed for RDP support in KRDC (at runtime)"
# NX support is not ready for KDE 4.2; disabled (uwolfer)
# macro_optional_find_package(LibNXCL)
# macro_log_feature(LIBNXCL_FOUND "libnxcl" "NX X compression client library" "" FALSE "1.0" "Needed to build Krdc with NX support")
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment