llama.cpp icon indicating copy to clipboard operation
llama.cpp copied to clipboard

ci : start using OpenSSL

Open angt opened this issue 1 month ago • 4 comments

angt avatar Nov 13 '25 11:11 angt

I think we should have CI dedicated to testing openssl instead of completely replace it.

The CI is suppose to validate the build with default config that will be pushed to end-user. If we aim to make openssl as default, we should change its default value in cmakelists instead.

ngxson avatar Nov 13 '25 12:11 ngxson

The idea is to remove completely the use of libcurl and all the code related. I'm waiting the boringssl PR to replace it everywhere :)

angt avatar Nov 13 '25 12:11 angt

The idea is to remove completely the use of libcurl and all the code related.

In this case, it makes more sense to the default value in cmakelists; probably also put up a deprecation warning when LLAMA_CURL is enabled

ngxson avatar Nov 13 '25 12:11 ngxson

The plan is to replace the current libcurl build with OpenSSL and BoringSSL (not only OpenSSL) first in the CI, then in all releases. If successful, I will deprecate the code by adding a warning message during the build process. And then changing the default.

I wouldn’t enable OpenSSL by default as a first step; this is guaranteed to make a lot of useless frictions.

angt avatar Nov 13 '25 12:11 angt