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

Missing Csocket source in 1.6.1 published tarball #1175

Closed
triplepoint opened this issue Nov 1, 2015 · 2 comments
Closed

Missing Csocket source in 1.6.1 published tarball #1175

triplepoint opened this issue Nov 1, 2015 · 2 comments

Comments

@triplepoint
Copy link

I might be misunderstanding the ZNC build process, but it seems that the third_party\Csocket directory is a git submodule, and that content is not making it into the published tarball at http://znc.in/releases/znc-1.6.1.tar.gz.

This manifests at build time, I think, as something like:

...
Building core object Csocket...
src/Csocket.cpp: In member function ‘virtual bool Csock::SSLClientSetup()’:
src/Csocket.cpp:1467:48: error: ‘SSLv3_client_method’ was not declared in this scope
   m_ssl_ctx = SSL_CTX_new( SSLv3_client_method() );
                                                ^
src/Csocket.cpp: In member function ‘SSL_CTX* Csock::SetupServerCTX()’:
src/Csocket.cpp:1589:43: error: ‘SSLv3_server_method’ was not declared in this scope
   pCTX = SSL_CTX_new( SSLv3_server_method() );

Maybe I misunderstood what's happening at build time, and there's something subtle about that third_party directory in the distributed tarball?

In any case, I've built an Ubuntu 14.04 virtual machine, and followed the instructions at http://wiki.znc.in/Installation#Build_From_Source and received the above error at make time.

I believe this same process worked fine about a month ago, if that's of any help.

@DarthGandalf
Copy link
Member

When packing tarball, Csocket.cc from the submodule is copied to src/Csocket.cpp with some modifications (see make-tarball.sh). Yes, it's subtle.

The SSL error is fixed at #1146 so you can use 1.7 from either git or nightly tarball

@triplepoint
Copy link
Author

A-ha, so I was barking up the wrong tree. Thanks for the quick feedback, I'll try the nightlies.

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