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

Emscripten support for BehaviorTree.CPP #765

Closed
wants to merge 1 commit into from

Conversation

coder137
Copy link

@coder137 coder137 commented Feb 10, 2024

Changes

  • Removed compile errors that crop up when compiling with emsdk 3.1.47
  • MiniCoro library errors
D:\CustomBuilds\emsdk\upstream\emscripten\cache\sysroot/include\emscripten/em_asm.h:152:1: error: templates must have C++ linkage
  152 | template<typename, typename = void> struct __em_asm_sig {};
      | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../3rdparty\minicoro/minicoro.h:310:1: note: extern "C" language linkage specification begins here
  310 | extern "C" {

Linked to issue: #679

Compiled on Windows with

  • emsdk activate
  • emcmake cmake -B build -G Ninja -DBTCPP_SHARED_LIBS=OFF -DBTCPP_BUILD_TOOLS=OFF -DBTCPP_UNIT_TESTS=OFF -DBTCPP_GROOT_INTERFACE=OFF -DBTCPP_SQLITE_LOGGING=OFF

@coder137 coder137 changed the title BehaviorTree.CPP support for Emscripten Emscripten support for BehaviorTree.CPP Feb 10, 2024
@facontidavide
Copy link
Collaborator

I think this is very good, but there are conflicts to be solved. Can you rebase and fix this?

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