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.

Sign up for GitHub

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

Jump to bottom

add cmake files to build and install header only library #85

Merged
merged 2 commits into from
Feb 2, 2021
Merged

add cmake files to build and install header only library #85

merged 2 commits into from
Feb 2, 2021

Conversation

ClausKlein
Copy link
Contributor

add simple cmake files to support cmake usage as subproject

optional support install and import of cmake config package files

Pre-merge checklist

  • I've read CONTRIBUTING.md
  • I've rebased my changes against the current HEAD of origin/master (if necessary)
  • I've added new test cases to verify my change
  • I've regenerated toml.hpp (how-to)
  • I've updated any affected documentation
  • I've rebuilt and run the tests with at least one of:
    • Clang 6 or higher
    • GCC 7 or higher
    • MSVC 19.20 (Visual Studio 2019) or higher
  • I've added my name to the list of contributors in README.md

@ClausKlein ClausKlein changed the title add cmake files to build and install header ohne library add cmake files to build and install header only library Feb 2, 2021
@marzer
Copy link
Owner

marzer commented Feb 2, 2021

Oh, nice! Thanks for this.

@marzer marzer merged commit dcfab29 into marzer:master Feb 2, 2021
yummyfs pushed a commit to yummyfs/tomlplusplus that referenced this pull request Feb 7, 2021
@ClausKlein @yummyfs
add cmake files to build and install header only library (marzer#85)
c11c8bc 
* add cmake build and install support

export cmake config package too

* enable bootstap build without cmake config package

cleanup cmake list file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ClausKlein @marzer