Chargement…
Référencer dans un nouveau ticket
Il n'existe pas encore de contenu.
Supprimer la branche '%!s(<nil>)'
Supprimer une branche est permanent. Cela NE PEUVENT être annulées. Continuer ?
Supprimer une branche est permanent. Cela NE PEUVENT être annulées. Continuer ?
Basic information
Description
Some distributions (like Void Linux) now provide LibreSSL instead of OpenSSL as a default. LibreSSL is OpenBSD's fork of OpenSSL 1.0.1b. TDElibs (especially KSSL in TDEIO) fails to compile.
excerpt from stdout: https://pastebin.com/37QfJMYc
Steps to reproduce
Screenshots
Please, what is the output from the CMake configuration that is related to SSL library detection?
stdout during configuration only mentions OpenSSL once:
CMakeCache.txt contains the following:
https://pastebin.com/n4kfEKH3
BTW, LibreSSL does provide some headers in /usr/include/openssl. I have LibreSSL 2.8.3 and in opensslv.h (which is imported in kopenssl.h) the macro which defines OpenSSL version number contains the value 0x20000000L:
https://pastebin.com/btthU22U
I think, with LibreSSL being a fork of OpenSSL 1.0.1b, that it is not compatible with some of the changes that newer versions brought about; though, OPENSSL_VERSION_NUMBER being > 0x10100000L, the compatibility checks in tdeio/kssl/kopenssl.h fail. I might as well be wrong.
I'm checking this right now and if I have results, I'll report back.
I was able to compile tdelibs making some changes to tdeio/kssl/kopenssl.h and tdeeio/kssl/ksslcertificate.cc
(some #ifndef LIBRESSL_VERSION_NUMBER's to check if the SSL flavour is LibreSSL)
My changes as a patch file: https://pastebin.com/e3mPVc1p
Hi Blu, thanks for the good work. Rather than posting a patch on pastebin, could you create a branch and a pull request here in gitea? That is the main advantage of using gitea, moreover authorship of the patch would be maintained automatically.
The wiki guide on TGW should provide some help on how to create a PR and if you need further help, just ask here.
@blu256: I have added you to the contributor group, so you can now create PRs 😃
Don't forget to sign-off your commit with the -s flag as per DCO.
Ok, done.
Thanks a lot and keep up the good work 😄
@blu256, thank you for your excellent cooperation.
Keep up the good work Philippe 😄
Hello,
there's a typo in the patch for ksslcertificate.cc which breaks compilation: you have written LIBERSSL_VERSION_NUMBER instead of LIBRESSL_VERSION_NUMBER
Thanks Stef. This has now been fixed. Could you please double check and let us know if ok?
I can't really do it. I was compiling Release 14.0.6 when I met the LibreSSL acceptance problem, then I found this patch and just copied-pasted the 3 relevant lines in the kopenssl.h and ksslcertificate.cc from 14.0.6, and it wouldn't compile because of the typo. Once corrected, it built OK.
So I can say it compiles with 14.0.6 (I can't say it works, since I haven't finished building the whole environment, but at least it builds OK), but I can't test the development version (I would need to recompile all dependencies with development versions).
Sorry.
But proofreading this 2 letters change seems fine ;-)
no problem. if you ever come around the same issue, just let us know. Thanks for poiting out the typo 👍