open-simulation-platform / libcosim

OSP C++ co-simulation library
https://open-simulation-platform.github.io/libcosim
Mozilla Public License 2.0
61 stars 10 forks source link

Update yaml-cpp (0.6.3 -> 0.7.0) #705

Closed markaren closed 2 years ago

markaren commented 2 years ago

Update YAML dependency. Requires a fairly recent conan version.

Edit: Also resolves #703

kyllingstad commented 2 years ago

Could this fix #703, by any chance?

markaren commented 2 years ago

Probably, I saw that your recent PR got the same error.

markaren commented 2 years ago

Nope, does not fix the compiler warning treated as error.

Produces warning C4275 on VS2019.

kyllingstad commented 2 years ago

Ok, they probably changed something upstream in yaml-cpp then. We may have to guard our yaml-cpp #includes with #warning directives to disable this specific warning. We probably shouldn't ignore it on a global level.