Skip to content

Add CMake support#2

Closed
GeorgH93 wants to merge 4 commits intolibstud:masterfrom
GeorgH93:cmake
Closed

Add CMake support#2
GeorgH93 wants to merge 4 commits intolibstud:masterfrom
GeorgH93:cmake

Conversation

@GeorgH93
Copy link
Copy Markdown

Hello, I really like your little UUID lib. Since I use CMake, I took the liberty to create a CMake build configuration for the project.

The CMake setup:

  • Builds a static library
  • Generates proper installation targets
  • Creates CMake config files for easy integration in other CMake projects
  • Maintains the existing directory structure
  • Generates the version header

I haven't tested it on all the platforms that are supported by the project, only Linux, Windows and MacOS.

@boris-kolpackov
Copy link
Copy Markdown
Contributor

Thanks for the PR, however, I am going to reject it, sorry.

I have a principled commitment to never touch CMake myself and I've been burned too many times by people contributing an alternative build system support or CI integration to one of my projects, then disappearing and leaving me to maintain it.

If you would like, you are welcome to fork this project and maintain CMake support there. I can then add a link to your fork to the project's README suggesting that people look for CMake support in your fork.

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.

2 participants