search

CesiumGS / cesium-unreal

Bringing the 3D geospatial ecosystem to Unreal Engine
https://cesium.com/platform/cesium-for-unreal/
Apache License 2.0
921 stars 295 forks source link

Need packaging tests for non-Windows platforms #1430

Open csciguy8 opened 4 months ago

csciguy8 commented 4 months ago

We'd like to catch potential user issues on other platforms too.

csciguy8 commented 4 months ago

Could fix issues like this before they get released, https://github.com/CesiumGS/cesium-unreal/issues/1427

kring commented 4 months ago

Indeed! That particular issue is interesting because the problem did get flagged during the plugin build, but it was only a warning and we never saw it. From the v2.5.0 build log:

Library '/Users/runner/work/cesium-unreal/cesium-unreal/packages/CesiumForUnreal/HostProject/Plugins/CesiumForUnreal/Source/CesiumRuntime/../ThirdParty/lib/iOS-xarm64/libktx.a' was not resolvable to a file when used in Module 'CesiumRuntime', assuming it is a filename and will search library paths for it. This is slow and dependency checking will not work for it. Please update reference to be fully qualified alternatively use PublicSystemLibraryPaths if you do intended to use this slow path to suppress this warning.