Skip to content

Added conan-based dependency management for the package.#37

Open
cwolfe2 wants to merge 1 commit into
nlohmann:masterfrom
cwolfe2:master
Open

Added conan-based dependency management for the package.#37
cwolfe2 wants to merge 1 commit into
nlohmann:masterfrom
cwolfe2:master

Conversation

@cwolfe2

@cwolfe2 cwolfe2 commented Apr 4, 2019

Copy link
Copy Markdown

No description provided.

@cwolfe2

cwolfe2 commented Apr 4, 2019

Copy link
Copy Markdown
Author
  • would need to modify appveyor environment to have conan installed
  • would need to have conan install step added, or done with external command in CMakeLists.txt
  • probably doesn't work out of the gate for x86, but x64 builds fine.
  • if you're interested, i can flesh out the other targets.

@nlohmann

nlohmann commented Apr 4, 2019

Copy link
Copy Markdown
Owner

I like the idea of having the possibility to use Conan, but it should not be a prerequisite. Is it possible to have a CMakeLists.txt that uses Conan if it is available, but falls back to the current behavior if not?

@cwolfe2

cwolfe2 commented Apr 4, 2019 via email

Copy link
Copy Markdown
Author

@nlohmann

nlohmann commented Apr 4, 2019

Copy link
Copy Markdown
Owner

Thanks a lot!

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