Skip to content
This repository was archived by the owner on Jul 26, 2024. It is now read-only.

Comments

Switch k4a and k4abt CMake to Module Mode#55

Merged
skalldri merged 9 commits intomicrosoft:melodicfrom
skalldri:module_mode
Aug 21, 2019
Merged

Switch k4a and k4abt CMake to Module Mode#55
skalldri merged 9 commits intomicrosoft:melodicfrom
skalldri:module_mode

Conversation

@skalldri
Copy link
Contributor

@skalldri skalldri commented Aug 21, 2019

Fixes #54

Description of the changes:

  • Add a CMake Find Module for k4a and k4abt, which provides a cross-platform way of locating the k4a libraries
  • Revamp CMake installation logic to install all dependencies of k4a, k4arecord, and k4abt
  • Bump CMake minimum version to 3.5 to match Ubuntu 16.04

Before submitting a Pull Request:

I tested changes on:

  • Windows
  • Linux

@skalldri
Copy link
Contributor Author

@RoseFlunder : this change adds better support for locating the various k4a libraries and dependencies, which should help out with your other changes!

It also adds automatic detection of the body tracking SDK: if discovered, it defines K4A_BODY_TRACKING, which you should be able to detect with

#if defined(K4A_BODY_TRACKING)

.... body tracking code goes here

#endif

@skalldri skalldri changed the title Search for k4a and k4abt CMake to Module Mode Switch k4a and k4abt CMake to Module Mode Aug 21, 2019
Copy link
Contributor Author

@skalldri skalldri left a comment

Choose a reason for hiding this comment

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

Passes build tests on Linux and Windows

@skalldri skalldri merged commit 54c76ff into microsoft:melodic Aug 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CMake should use Module Mode instead of Config Mode

1 participant