-
-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
cocoapods integration #1150
cocoapods integration #1150
Conversation
Is it possible to put the source files in a different directory? |
yes where do you want the dummy source files to be? |
interesting dummy files aren't needed. I was using an older version of cocoapods. It seems cocoapods linter is passing with just header only. |
nlohmann_json.podspec
Outdated
# ――― Spec Metadata ―――――――――――――――――――――――――――――――――――――――――――――――――――――――――― # | ||
s.name = "nlohmann_json" | ||
s.version = "3.1.2" | ||
s.summary = "JSON reader & writer for C++ optimized for ease of use and modern C++14" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's not really the summary of the library. Please use "JSON for Modern C++" or something. And the library also supports C++11.
nlohmann_json.podspec
Outdated
|
||
# CocoaPods strips indentation for description | ||
s.description = <<-DESC | ||
JSON library for c++, it can read & write, uses C++14 features to have |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See previous comment.
nlohmann_json.podspec
Outdated
|
||
|
||
s.authors = { | ||
"Niels Lohmann" => "nosend@respectmyprivacy.com" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My email address is mail@nlohmann.me.
nlohmann_json.podspec
Outdated
# the deployment target. You can optionally include the target after the platform. | ||
# | ||
|
||
s.platform = :ios, "8.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
macOS should also be fine, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I only tested iOS, theoretically it should be fine on macOS, watch and all other platforms. I don't want to add untested platforms.
adds cocoapods integration so it can be used as a pod. See issue #1148