Sanjivkumarroshan / csipsimple

Automatically exported from code.google.com/p/csipsimple
0 stars 0 forks source link

New version of openssl fork #1285

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Would be nice to see a newer version of OpenSSL to be included.

Current version seems to be copied from 
http://github.com/fries/android-external-openssl

A newer fork can be found on:
https://github.com/eighthave/openssl-android

Original issue reported on code.google.com by domschuermann@gmail.com on 23 Sep 2011 at 3:07

GoogleCodeExporter commented 9 years ago
Thanks for the hint. I'll have a look to the new fork.

Actually I did modifications in build process of fries' version. 
And in fact android port are usually mostly about build process ;). 
So, not sure improvements are relevant for pjsip_android project. But if there 
is interesting patches or updates of openssl I'll update that in the trunk.

Original comment by r3gis...@gmail.com on 14 Oct 2011 at 1:58

GoogleCodeExporter commented 9 years ago
Thanks for looking into it.
They moved their repository to 
https://github.com/guardianproject/openssl-android

Original comment by domschuermann@gmail.com on 14 Oct 2011 at 3:08

GoogleCodeExporter commented 9 years ago
It will be included in future releases. It will now checkout the git repo from 
guardianproject.

However, I'm wondering if I should include libssl and libcrypto in a stock 
build. Since it will be available on 99.9% of android builds, it may be 
ununecessary to ship the openssl lib into the application itself. I think that 
trying to rely on stock shared lib if available and to fallback to a library 
shipped into a "plugin" app would be a good idea. 
What do you think? 
Is there some benefit to try to load the re-built openssl library over the 
android's one? All the more so as, I'm not absolutely sure it actually load it 
over the stock one on all android OS versions (dyn loader seems to behaves 
differently on 1.6 for example).

Original comment by r3gis...@gmail.com on 29 Oct 2011 at 6:49

GoogleCodeExporter commented 9 years ago
If it is possible to load the openssl library from the android os, this would 
be nice to reduce size of CSipSimple.

Unfortunately I don't know how this can be done or how Android behaves in this 
case. I found a Google groups post about something similar. Eventually it helps:
http://groups.google.com/group/android-ndk/browse_thread/thread/76d62fde6705c5f6

Original comment by domschuermann@gmail.com on 30 Oct 2011 at 11:58

GoogleCodeExporter commented 9 years ago
Included in nightly build. Only useful to build the app however. [*]
If possible relies on the library of the phone. Tested on 1.6, 2.1, 2.2, 2.3 
and 3.0 and seems to work correctly.

[*] : all is delegated to stock shared library except one little part of the 
crypto lib that is embedded into the app because not available on android 2.2 
only. 

Original comment by r3gis...@gmail.com on 24 Nov 2011 at 6:29