Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Conan 2 #64

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Use Conan 2 #64

wants to merge 11 commits into from

Conversation

johannes-wolf
Copy link
Contributor

@johannes-wolf johannes-wolf commented Dec 13, 2023

Use Conan 2 for package management.

Emscripten + Node.js get installed via Conan. There is currently a problem of Conan packages not allowing to override their tool-requirements, so we install nvm and node.js v21 in the build-ui.bash script into a temporary directory to force override the node version Conan installs for us (which is too old for angular). See: conan-io/conan-center-index#23088.

This also fixes problems with conflicting CMake targets from cesium by building cesium as an external project (see cmake/cesium.cmake).

The cross-build is set up using a Conan profile.

  • Publish mapget to conan.io and use it from there

@johannes-wolf johannes-wolf changed the title WIP [WIP] Use Conan Dec 13, 2023
@johannes-wolf johannes-wolf force-pushed the use-conan branch 12 times, most recently from 5c05042 to 2d45839 Compare December 13, 2023 21:50
@johannes-wolf johannes-wolf changed the title [WIP] Use Conan Use Conan Dec 13, 2023
@johannes-wolf johannes-wolf changed the title Use Conan Use Conan 2 Dec 13, 2023
@johannes-wolf johannes-wolf force-pushed the use-conan branch 2 times, most recently from 55b5dc3 to c0154f7 Compare December 16, 2023 15:07
@johannes-wolf johannes-wolf force-pushed the use-conan branch 9 times, most recently from d51500c to 3ec21d8 Compare March 8, 2024 18:03
@johannes-wolf johannes-wolf force-pushed the use-conan branch 9 times, most recently from c8668c2 to 73c7d4d Compare March 15, 2024 13:26
@johannes-wolf
Copy link
Contributor Author

Waiting for emsdk linux64 build clarification.

@johannes-wolf
Copy link
Contributor Author

@Waguramu @josephbirkner, do we have some (new) minimum version requirements on emsdk Node.js durring build? These build errors appeared after rebasing this PR on master.

@josephbirkner
Copy link
Collaborator

@Waguramu @josephbirkner, do we have some (new) minimum version requirements on emsdk Node.js durring build? These build errors appeared after rebasing this PR on master.

Yes, we upgraded to 3.1.56 due to this: emscripten-core/emscripten#21460

@johannes-wolf
Copy link
Contributor Author

Ok. Conan has no emsdk > 3.1.50 yet, so we either have to wait or get conan to use the hosts emsdk. I will have a look into the second option later; there is something like [system_requires].

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants