libsrtp icon indicating copy to clipboard operation
libsrtp copied to clipboard

update config options for crypto backend

Open pabuhler opened this issue 1 year ago • 2 comments

Wrong things will happen if you enable two or more of --enable-openssl, --enable-wolfssl, and --enable-nss, and I think we have nothing that guards against this. This isn't a new problem with wolfssl (it was already a problem with just openssl and nss), but should we have something to protect against this?

Or possibly even change the configure option syntax to --enable-crypto=[openssl,wolfssl,nss] with the 3.0 release, since that's where we're putting other breaking changes.

Originally posted by @JonathanLennox in https://github.com/cisco/libsrtp/issues/692#issuecomment-2026406283

pabuhler avatar Apr 01 '24 20:04 pabuhler

I am happy to add this feature in automake, meson, and Cmake. There is no option for me to create branch. @pabuhler Can you please help me here.

~Sudhir

SudhirKesti avatar Dec 21 '24 01:12 SudhirKesti

@SudhirKesti , the normal way to do this is to fork the repo and there create a branch in the fork. After wards you can create a PR to merge that branch in to our repo. This is a pretty standard way of doing things, if you search online I am sure you will find some better explanations.

pabuhler avatar Dec 21 '24 20:12 pabuhler