search

CesiumGS / cesium-unreal

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

Add missing header for some builds #1437

Closed csciguy8 closed 1 month ago

csciguy8 commented 1 month ago

Closes #1434.

I repro'd this error by doing this: 1) Downloaded CesiumForUnreal-53-v2.5.0.zip from the releases page 2) Open Unreal 5.3, create a new Blueprints project 3) Unzip the plugin to this projects "Plugins" folder (I had to create this) 4) Open the test project, enable the Cesium For Unreal plugin 5) Close Unreal 6) Delete the contents of <testProject>/Plugins/CesiumForUnreal/Binaries 7) Open the test project again, it should ask you to rebuild 8) When it finishes, you'll see a popup dialog that says there were errors (you can see details in <testProject>/Saved/Logs

Using the build from this PR should fix this error.

csciguy8 commented 1 month ago

Some other notes: 1) I'm not sure how this kind of build is different from our normal plugin builds we use for our release. (header order differences?) 2) I'm not sure why the original user's case forced a build to happen. In my local test I had to forcibly delete the binaries to get the plugin to rebuild.

kring commented 1 month ago

Thanks @csciguy8!