properly show xfreerdp runtime dependency in cmake summary
ClosedPublic

Authored by sitter on Jan 10 2019, 11:39 AM.

Details

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

Diff Detail

Repository
R436 KRDC
Branch
Applications/18.12
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 6903
Build 6921: arc lint + arc unit
sitter requested review of this revision.Jan 10 2019, 11:39 AM
sitter created this revision.
sitter added a subscriber: whiting.

somewhat unrelated ... @whiting do you recall why line 83 still says "Needs porting"? according to the git history you ported this a couple months after but the comment was not removed.

aacid added a subscriber: aacid.Jan 10 2019, 7:34 PM
aacid added inline comments.
CMakeLists.txt
95

I know the binary name is xfreerdp, but it's not the name they use themselves on the web and and it's not the name of the package distributions have either, i'd be more confortable with freerdp, but feel free to disagree

sitter added inline comments.Jan 11 2019, 9:18 AM
CMakeLists.txt
95

The motivation behind that was to communicate that we need that specific binary. As a compromise we could call the package freerdp and simply mention the binary in the PURPOSE I guess? That way we get both bits of information right.

sitter updated this revision to Diff 49229.Jan 11 2019, 11:32 AM

rename the package to freerdp but explicitly mention that the xfreerdp binary is needed

aacid accepted this revision.Jan 11 2019, 6:12 PM
This revision is now accepted and ready to land.Jan 11 2019, 6:12 PM
This revision was automatically updated to reflect the committed changes.
sitter marked an inline comment as done.