EduardoRFS / reason-mobile

MIT License
174 stars 10 forks source link

Use static OpenSSL for @opam/ssl #17

Open EduardoRFS opened 3 years ago

EduardoRFS commented 3 years ago

Problem

@opam/ssl just add -lssl -lcrypto, so if there is a libssl.so and libcrypto.so available it will use it, instead of the static.

This is a problem because then applications needs to ship the shared one, and for platforms like Android even patchelf is needed.

Workaround

Remove the shared library version, like #16

Proper possible solution

Add a flag to use the static version on discover.ml#L17 and add a patch on reason-mobile so that it always uses the static one.