Skip to content
This repository has been archived by the owner on Feb 19, 2021. It is now read-only.

Switch to cmake build system #191

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

MatejFranceskin
Copy link
Contributor

This is the draft version of cmake build system for camera manager.
I would like to get the opinion from the project owners about the task and recommendations about what and how should it be implemented.

@MatejFranceskin MatejFranceskin marked this pull request as ready for review March 9, 2019 17:55
Copy link
Contributor

@julianoes julianoes left a comment

Choose a reason for hiding this comment

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

I made some comments but I think this is generally looking good!

And I prefer it over configure mostly because I know it better.

.gitignore Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
cmake/FindAvahi.cmake Show resolved Hide resolved
src/CMakeLists.txt Outdated Show resolved Hide resolved
src/PluginBase.h Show resolved Hide resolved
src/Plugins/AeroAtomIspCamera/CMakeLists.txt Outdated Show resolved Hide resolved
src/Plugins/GazeboCamera/CMakeLists.txt Outdated Show resolved Hide resolved
src/Plugins/RealSenseCamera/CMakeLists.txt Outdated Show resolved Hide resolved
test/CMakeLists.txt Outdated Show resolved Hide resolved
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.

2 participants