Summary: | ASTERISK-06321: [patch] Asterisk binary linked with openssl | ||
Reporter: | Denis Smirnov (mithraen) | Labels: | |
Date Opened: | 2006-02-15 07:28:12.000-0600 | Date Closed: | 2008-01-15 16:50:30.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
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) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=10323 |