Hi Willy,

>> Like previsouly suggested by Apollon, we could introduce a flag for
>> statically linking openssl, like we have with PCRE (USE_STATIC_OPENSSL?).
>>
>> If we do that, we could also include -ldl automatically when using that USE
>> flag.
>>
>> Thoughts?
>
> I'm a bit hesitant here and am not completely fond of what we did for
> PCRE_STATIC. The problem is that we force -Wl,-Bstatic then -Wl,-Bdynamic,
> and that it breaks full static builds. There's no option in the linker
> like "-Wl,-Bdefault" to say "let's come back to the default model", so
> we have to force "dynamic" by hand at the end of inclusion of the lib
> and complicate the things.

I see; I was not aware of this problem.



> Or maybe we could have a variable "BUILD_MODEL" = {static,dynamic} that
> we use to set the fallback mode and take the correct decisions when adding
> libraries.

You mean maintaining the USE_STATIC_LIBXY approach and use the BUILD_MODEL
variable to fix this problem? Could be a possibility, but its gets more
complex again; perhaps we should rethink the whole USE_STATIC_LIBXY approach
instead? Not sure.



> Anyway, I tend to prefer to let people pass an option to fix their build
> issues than having a few other ones patch the makefile to get rid of
> something that was added for laziness and breaks their build.

Agreed.



Sending the README patch about SSL_INC/SSL_LIB now.



Regards,

Lukas                                     

Reply via email to