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

Published 20 hours ago • verified publisher icon CesiumGS

Conan!

Open kring opened this issue 3 years ago • 22 comments

This PR does a bunch of things:

  • Drastically simplifies the CMake setup:
    • Extracted all the utility/helper CMake code to tools/cesium.cmake.
    • Each library's CMakeLists.txt is now a proper CMake project that can be used standalone.
    • The CMake code in each configuration has been drastically simplified. It mostly just calls cesium_library with its name and dependencies. That function takes care of everything else.
    • Our CMake still does not depend on Conan at all. Dependencies are found by using find_package.
  • Eliminates all the git submodules.
  • Adds a cesium-native.yml file in the root, and a library.yml in each library directory. These files capture information about cesium-native and the libraries that it contains, including third-party dependencies and their versions.
  • Added tools/automate.py to do various things, like generate Conan recipes for the libraries from the descriptors in library.yml and cesium-native.yml. Strangely enough this is the first Python program I've ever written, so hopefully it's not too terrible. I used Python because Conan is pretty all-in on it, and it was pretty easy to do. See README.md.
  • Switched from modp_b64 (copied out of Chromium) to base64 (from Conan Center).
  • Added recipes for the three remaining third-party libraries that are not yet on Conan Center: csprng, picosha2, and s2geometry.
  • Added VSCode CMake Kits to use toolchain files generated by Conan.
  • Added instructions for installing dependencies with Conan and building Conan packages.

Fixes #230 Fixes #248

This PR is a draft until there's a rock-solid cesium-unreal PR to go with it.

kring avatar Feb 23 '22 01:02 kring

looks good to me! Nothing comes out as odd

baothientran avatar Mar 01 '22 18:03 baothientran

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Apr 01 '22 02:04 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar May 01 '22 02:05 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar May 31 '22 02:05 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Jun 30 '22 02:06 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Jul 30 '22 02:07 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Aug 29 '22 02:08 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Sep 28 '22 02:09 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Oct 28 '22 02:10 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Nov 27 '22 03:11 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Dec 27 '22 03:12 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Jan 26 '23 03:01 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Feb 25 '23 03:02 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Mar 27 '23 02:03 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Apr 26 '23 02:04 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar May 26 '23 02:05 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Jun 25 '23 02:06 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Jul 25 '23 02:07 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Aug 24 '23 02:08 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Sep 23 '23 02:09 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Oct 23 '23 02:10 cesium-concierge

Thanks again for your contribution @kring!

No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?

I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

cesium-concierge avatar Nov 22 '23 03:11 cesium-concierge