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

Use system-provided nlohmann_json if available #87

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cottsay
Copy link
Contributor

@cottsay cottsay commented May 16, 2020

Otherwise, maintain current behavior of using the bundled version.

Otherwise, maintain current behavior of using the bundled version.
@pietern
Copy link
Owner

pietern commented Feb 13, 2022

This is a header-only library, so I don't see how using the system provided version would be needed.

Were you trying to get a build working without the vendor directory (submodules)?

@cottsay
Copy link
Contributor Author

cottsay commented Feb 14, 2022

Thanks for triaging all of these old tickets, @pietern.

Were you trying to get a build working without the vendor directory (submodules)?

Indeed I was. I'd love to see this package available as an OS package someday, and some platforms look poorly on bundled software. My goal was to allow use of the system package while still supporting the bundling approach you prefer as the default.

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