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

NUP-2320 DO NOT MERGE Convert nupic.core to shared lib. #1216

Conversation

vitaly-krugl
Copy link
Member

@vitaly-krugl vitaly-krugl commented Jan 24, 2017

Fixes #1217.

Work in progress ...

Vitaly Kruglikov added 30 commits February 1, 2017 12:48
…ror from `import nupic.bindings.engine_internal`:

ImportError: dlopen(/Users/vkruglikov/Library/Python/2.7/lib/python/site-packages/nupic/bindings/_engine_internal.so, 2): Library not loaded: libnupic_core.dylib
  Referenced from: /Users/vkruglikov/Library/Python/2.7/lib/python/site-packages/nupic/bindings/_engine_internal.so
  Reason: unsafe use of relative rpath libnupic_core.dylib in /Users/vkruglikov/Library/Python/2.7/lib/python/site-packages/nupic/bindings/_engine_internal.so with restricted binary

In [3]: import nupic.bindings.engine_internal
NUP-2320 Fix installation of nupic_core dll into the prefix/bin directory so
that it works on Windows as well, since there it's treated as RUNTIME instead of LIBRARY.
…(needed by PyRegion).

Added temp `ls` statement to help debug why libnupic_core.so isn't being found in most Bamboo build targets.
…hared lib to force generation of capnp schema.
…nto nupic_core shared lib that don't have explicit visibility to avoid prining when building on Windows.
…ared

library containing capnproto to satisfy capnp references that would normally
be satisfied by capnproto in pycapnp.
…or Linux and mingw (was failing on mingw without -Wl)
NUP-2320 Fix reference to src_common_os_libs variable.
…RE in src_common_test_exe_libs.

NUP-2320 experimentally, comment out the PRIVATE keyword in target_link_libraries of shared libs.
Vitaly Kruglikov added 2 commits February 1, 2017 12:48
@vitaly-krugl vitaly-krugl force-pushed the nup2320-confert-nupic.core-to-sharedlib branch from 1306982 to 0e5d18e Compare February 1, 2017 21:00
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.

1 participant