CesiumGS / cesium-native

Apache License 2.0
391 stars 200 forks source link

Add support for static linking with MSVC runtime library #907

Closed csciguy8 closed 3 weeks ago

csciguy8 commented 1 month ago

Based off of a community contribution from #900 (thanks again @jqntn!).

Creates a new CMAKE option to specify dynamic or static VC runtime linkage for windows platforms.

For unit tests, we make use of this new option, which fixes a recent github build runner issue, and closes #906.

Supports https://github.com/CesiumGS/cesium-unity/issues/419, but doesn't close it.

csciguy8 commented 1 month ago

@kring, you wrote up the linked cesium-unity issue, so you may be interested in reviewing this.

kring commented 3 weeks ago

Thanks @jqntn and @csciguy8!