search

pelicanmapping / rocky

3D Geospatial SDK (C++17 / Vulkan / VSG)
MIT License
91 stars 15 forks source link

fix: more clang compile/link errors. #11

Closed remoe closed 1 year ago

remoe commented 1 year ago

Not sure about the lambda captures. clang throw "Non-local lambda expression cannot have a capture-default"

tests doesn't compile (temp excluded from build):

/usr/include/c++/v1/__random/uniform_int_distribution.h:235:5: Static_assert failed due to requirement '__libcpp_random_is_valid_urng<Catch::RandomNumberGenerator, void>::value' "" rocky/src/tests/catch.hpp:1630:49: Use of overloaded operator '<<' is ambiguous (with operand types 'std::ostream' (aka 'basic_ostream') and 'const rocky::optional')

Adding "nlohmann_json::nlohmann_json" to target_link_libraries could be a cmake issue. I don't use VCPKG, because it doesn't work really on other platforms then windows.

gwaldron commented 1 year ago

Remo, I incorporated most of these changes and pushed them separately. I did not include your build-specific changes for now. Best to keep those local until we can get GitHub Actions functioning for all platforms.