cesium-native icon indicating copy to clipboard operation
cesium-native copied to clipboard

Allow a way to specify an external sqlite3 library

Open keyboardspecialist opened this issue 2 years ago • 3 comments

Currently cesium-native uses its own internal sqlite3 lib. We would like a way to specify our own version from sources such as Conan, so all of our dependencies are aligned across each project.

keyboardspecialist avatar May 23 '23 15:05 keyboardspecialist

To add more context:

PRIVATE_CESIUM_SQLITE was added to the cmake build to namespace against Unreal's include sqlite. However, this doesn't protect internal projects from having linker conflicts.

Pointing CesiumAsync to the Conan sqlite is a viable solution though we need to ensure that the cesium-sqlite.h header still works as we expect.

keyboardspecialist avatar May 23 '23 17:05 keyboardspecialist

However, this doesn't protect internal projects from having linker conflicts.

Are you sure? It should. When that is defined, every symbol in SQLite gets renamed.

You'll end up with two copies of SQLite, which may not be ideal, but it should work.

kring avatar May 23 '23 23:05 kring

However, this doesn't protect internal projects from having linker conflicts.

Are you sure? It should. When that is defined, every symbol in SQLite gets renamed.

You'll end up with two copies of SQLite, which may not be ideal, but it should work.

You are correct. I realized the flag wasn't getting properly set. Builds fine once turned on.

keyboardspecialist avatar May 24 '23 15:05 keyboardspecialist

It seems like this issue was resolved, so I will close the issue.

azrogers avatar Jul 03 '25 19:07 azrogers