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

Fix MacOS compilation, add libsecure to target link libs for nano_lib #3947

Conversation

dsiganos
Copy link
Contributor

The problem appears to be that nano_lib uses libsecure but it is not listing it as a dependency. The ipc_flatbuffers_test_client does not list libsecure and the compilation was failing.

The problem was only happening in MacOs Debug builds only. I assume that it was happening in Debug builds only because of an assert that used libsecure code.

However, I do not understand why this problem was not appearing on linux.

@dsiganos dsiganos force-pushed the fix_macos_compilation_ipc_flatbuffers_test_client branch from 38fca9f to e494e4b Compare September 13, 2022 09:38
pwojcikdev
pwojcikdev previously approved these changes Sep 13, 2022
The problem appears to be that nano_lib uses libsecure but it is not
listing it as a dependency. The ipc_flatbuffers_test_client does not list
libsecure and the compilation was failing.

The problem was only happening in MacOs Debug builds only.
I assume that it was happening in Debug builds only because of an assert
that used libsecure code.

However, I do not understand why this problem was not appearing on linux.
@dsiganos dsiganos force-pushed the fix_macos_compilation_ipc_flatbuffers_test_client branch from e494e4b to 256801f Compare September 13, 2022 10:49
@dsiganos dsiganos merged commit 9011cef into nanocurrency:develop Sep 13, 2022
@dsiganos dsiganos deleted the fix_macos_compilation_ipc_flatbuffers_test_client branch September 13, 2022 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants