Skip to content
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

Reworked CMakeLists.txt #121

Merged
merged 1 commit into from
Mar 20, 2024
Merged

Reworked CMakeLists.txt #121

merged 1 commit into from
Mar 20, 2024

Conversation

kobalicek
Copy link
Contributor

  • Moved the logic into a single CMakeLists.txt file
  • Added miniocpp:: namespace and thus miniocpp::miniocpp target
  • Removed the generation of header files (config.h.in)
  • Consolidated tests and examples - use MINIO_CPP_TEST to build them all
  • Use miniocpp::miniocpp target in tests and exampels to simplify the use of dependencies
  • Setup the cmake project with DESCRIPTION and VERSION
  • Use GNUInstallDirs when installing targets
  • Export cmake files so another cmake can find minio-cpp by using find_package(miniocpp) and then use miniocpp::miniocpp
  • Building both documentation and tests/examples is OFF by default

@kobalicek kobalicek force-pushed the cmake_update branch 5 times, most recently from e8b4871 to 1c48cb3 Compare March 19, 2024 16:24
  * Moved the logic into a single CMakeLists.txt file
  * Added miniocpp:: namespace and thus miniocpp::miniocpp target
  * Removed the generation of header files (config.h.in)
  * Consolidated tests and examples - use MINIO_CPP_TEST to build
    them all
  * Use miniocpp::miniocpp target in tests and exampels to simplify
    the use of dependencies
  * Setup the cmake project with DESCRIPTION and VERSION
  * Use GNUInstallDirs when installing targets
  * Export cmake files so another cmake can find minio-cpp by
    using find_package(miniocpp) and then use miniocpp::miniocpp
  * Building both documentation and tests/examples is OFF by default
@harshavardhana harshavardhana merged commit b29b43e into main Mar 20, 2024
3 checks passed
@harshavardhana harshavardhana deleted the cmake_update branch March 20, 2024 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants