-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Discussion: Move to CMake? #12
Comments
Hi Ben, Please briefly describe your problems and modifications. thanks! |
Yeah, the CMake setup would involve a lot of cross-dependencies, but that would make it easier to use in the future. For example, we use most libraries as git submodules, and just do For proper CMake usage I'd create a central CMake that includes all the sub-projects and allows users to specify which libraries they want build or made available to their software. |
Hi Ben, There are some competing goals for us when it comes to making things simple. "simple" depends on the audience. |
My Robotics team (UTK IEEE) decided to buy a Pixy 2 to do colored object recognition for this year's SoutheastCon challenge, and we've found it somewhat difficult to integrate this library into other projects.
We're working with a large C++ codebase in which we extensively use CMake for inter-repo dependencies and to improve platform interoperability.
We hacked together a fork just so that we could use the libpixyusb2 sources, but it's very much a hack. If there's interest, we'd like to contribute some work back to upstream to perhaps reorganize the source code here to make it easier to include and use.
Would changes moving almost all of the source code be welcome? It'd probably restructure all of the current directory layout to make it clearer how the dependency tree works out, and to provide better organization for including specific sub-libraries via CMake usage.
Forked (hackish) version: https://gitlab.com/utk-robotics/forks/pixy2
The text was updated successfully, but these errors were encountered: