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

Tox libraries are not linked properly to tox.so #17

Closed
gordon-quad opened this issue Apr 13, 2018 · 5 comments
Closed

Tox libraries are not linked properly to tox.so #17

gordon-quad opened this issue Apr 13, 2018 · 5 comments

Comments

@gordon-quad
Copy link
Contributor

After commit 1cc1aa2 make produced tox.so which is not linked to libtoxav.so and libtoxencryptsave.so which leads to inability to load tox.so in weechat:

unable to load plugin "/home/gordon/.weechat/plugins/tox.so": /home/gordon/.weechat/plugins/tox.so: undefined symbol: tox_pass_encrypt

I am using c-toxcore from git and it seems that it never produced single libtoxcore.so with everything linked inside. I wonder why did you change FindTox.cmake to exclude those libraries from Tox_LIBRARIES cmake variable?

@gordon-quad
Copy link
Contributor Author

After i did the following fix 78b0621 it works for me now

@gordon-quad gordon-quad changed the title Tox libraries is not linked properly to tox.so Tox libraries are not linked properly to tox.so Apr 13, 2018
@haavard
Copy link
Owner

haavard commented Apr 13, 2018

That's my bad -- I assumed the newer version of toxcore built a monolithic shared object, but it looks like it depends on whether or not toxcore is built with cmake or autotools. Our CMake find module for Tox will have to work for both of these situations.

@gordon-quad
Copy link
Contributor Author

FYI TokTok/c-toxcore#869

@gordon-quad
Copy link
Contributor Author

According to conversation in IRC cmake is preferred way of building toxcore and autotools script needs updating, so I suppose its time to fix gentoo ebuilds and get single libtoxcore.so.

But since they are not going to fix autotools script soon it still would be nice to support both monolithic and split versions in FindTox.cmake if it will not be too difficult to implement.

haavard added a commit that referenced this issue Apr 30, 2018
toxcore may be built as either a single monolithic library, or multiple
libraries (core, AV, encrypted savefiles). This updates the CMake find
module to work in both configurations.

Fixes <#17>.
@haavard
Copy link
Owner

haavard commented Apr 30, 2018

Should be fixed in the commit above.

@haavard haavard closed this as completed Apr 30, 2018
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

No branches or pull requests

2 participants