-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
[pdal] Prepare for nlohmann-json 3.11.x #26206
[pdal] Prepare for nlohmann-json 3.11.x #26206
Conversation
In nlohmann-json 3.11.0, json_fwd.hpp changed. Replace PDAL's internal copy of the file as part of the de-vendoring process.
207cdcd
to
39dbc6c
Compare
There's no mention of nlohmann-json in pdal's |
I'm confused about that as well, but it clearly references And |
I've no experience with
I could go on but let's start here. |
nlohman-json dependency is implicitly satisfied: (pdal -> libgeotiff -> nlohman-json). But it should be explicit. |
Right. I must have been looking at the wrong file. Thanks for clarifying, I'll add I'm not a fan of conflating the bundled single-header |
39dbc6c
to
bef84bb
Compare
Thanks! |
In nlohmann-json 3.11.0, json_fwd.hpp changed.
Replace PDAL's internal copy of the file as part of the de-vendoring process.
This PR is a requirement for #26124 to move forward.
Which triplets are supported/not supported? Have you updated the CI baseline?
No change, No
Does your PR follow the maintainer guide?
Yes
If you have added/updated a port: Have you run
./vcpkg x-add-version --all
and committed the result?Yes