Summary:ASTERISK-06321: [patch] Asterisk binary linked with openssl
Reporter:Denis Smirnov (mithraen)Labels:
Date Opened:2006-02-15 07:28:12.000-0600Date Closed:2008-01-15 16:50:30.000-0600
Versions:Frequency of
Environment:Attachments:( 0) openssl.patch
Description:Only res_crypto.so must be linked with openssl, but now all binaries linked with openssl.
Comments:By: Clod Patry (junky) 2006-02-15 10:19:27.000-0600

-1 for me.

By: Russell Bryant (russell) 2006-02-15 11:44:52.000-0600

Instead of adding it for everything but Linux, I'd rather try to find out which platform (if any) requires it, and only add it in those cases.

By: Denis Smirnov (mithraen) 2006-02-15 11:54:59.000-0600

I think that this behavior would be better then current behavior.
This patch:
- make things better for linux;
- don't make things worse for anyone

By: Russell Bryant (russell) 2006-02-15 14:27:26.000-0600

I certainly understand these statements.  However, I think there is more value in taking the time to figure out the complete correct solution for an issue up front, instead of having to re-visit it again later.

By: Denis Smirnov (mithraen) 2006-02-16 09:57:06.000-0600

Ok. Do you know any OS that have problems if binary and modules linked with not same libraries?

I don't know anyone.

Who can have this information, or where I can get this?

By: Russell Bryant (russell) 2006-02-16 10:45:38.000-0600

Looking back through the subversion history, this was initial added in revision 3297, as a part of adding OSP support.  I will ask Mark to see if he remembers why this was done.

By: Russell Bryant (russell) 2006-02-16 13:38:47.000-0600

After discussing this with Mark, we decided that we could remove it from Asterisk all together.  Thanks!

By: Digium Subversion (svnbot) 2008-01-15 16:50:30.000-0600

Repository: asterisk
Revision: 10323

U   trunk/Makefile
U   trunk/res/Makefile

r10323 | russell | 2008-01-15 16:50:30 -0600 (Tue, 15 Jan 2008) | 2 lines

remove -lssl for asterisk, and add it for res_osp (issue ASTERISK-6321, different patch)