-
-
Notifications
You must be signed in to change notification settings - Fork 149
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
fails to build when tox-core was built with nacl instead of libsodium #31
Comments
tox-core config.log https://gist.github.com/hasufell/db0e96d9fbac7bbe2e9a |
Mind trying this again, but compile toxcore as a static library? |
Could this be an issue similar to #47? |
Mind testing this again? So I know if it can be closed. |
Is this solved? |
FYI, I use the following patch in Tox to build venom correctly with NaCl in FreeBSD: --- patch-libtoxcore.pc.in begins here --- --- patch-libtoxcore.pc.in ends here --- But this does'nt work with Toxic. |
I did some tuning to the core, basically I was told that when core is built vs nacl, the shared version of core should be disabled. I fixed pkg-config (i.e. also added NACL_LIBS) and set up core's configure in a way that it will automatically disable shared-library build if nacl is enabled. Does it still fail with today's combination of core/toxic? I was trying to set up a BSD vm so I could have a look but did not succeed at getting BSD running properly/setting up a dev environment, if you have a shell for me where I could test, I'd appreciate it. |
On Wed, Mar 19, 2014 at 02:20:24PM -0700, Thierry Thomas wrote:
Uhm.. this gist is empty... are you sure you got the correct link? |
I removed it with my answer - but too late since you got it! I found the reason: Tox-Core's API changed, and it must rebuilt too. |
Sorry for the delay... Here are the results of my tests. With `pkg-config --libs libtoxcore' returning: -L/usr/local/lib -ltoxcore -lnacl /usr/local/lib/cpucycles.o /usr/local/lib/randombytes.o Venom links fine, but toxic fails with the message: --- toxic --- When `pkg-config --libs libtoxcore' only returns: -L/usr/local/lib -ltoxcore -lnacl toxic builds fine, and Venom still builds! |
Thanks, I think I now know what the problem is, those static files should not appear in the .pc file at all, I'll submit a PR to toxcore to fix that. |
I did a few more tests, basically it seems that on linux the object files are needed when linking applications vs. a nacl-based toxcore, at the same time this seems to cause linking errors on freebsd systems. Toxcore configure script will be tuned in the way that it will add the object files to the .pc on linux but not do so on freebsd, this should hopefully resolve this issue. Pleae test as soon as the toxcore PR gets merged: irungentoo/toxcore#815 |
It's OK. But I have a question: why did you disable the shared lib in tox when NaCl is enabled? |
On Tue, Mar 25, 2014 at 03:12:36PM -0700, Thierry Thomas wrote:
Actually I had errors when building shared toxcore libs vs. a static NaCl My tests and observations confirmed that. Correct me if I'm wrong... |
During my previous tests on FreeBSD, I've been building a shared lib for tox, and venom was OK. |
On Tue, Mar 25, 2014 at 03:43:17PM -0700, Thierry Thomas wrote:
I'll try again, but I mainly followed the suggestion in the channel since I |
build log https://gist.github.com/hasufell/46aa2b38052955452a86
The text was updated successfully, but these errors were encountered: