Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Are versions in util/libssl.num supposed to increment alongside minor version? #17516

Closed
satmandu opened this issue Jan 14, 2022 · 1 comment
Closed

Are versions in util/libssl.num supposed to increment alongside minor version? #17516

satmandu opened this issue Jan 14, 2022 · 1 comment
Labels
resolved: answered The issue contained a question which has been answered triaged: question The issue contains a question

Comments

@satmandu
Copy link

openssl/util/libssl.num

Line 512 in b4e83ed

SSL_CTX_load_verify_dir 513 3_0_0 EXIST::FUNCTION:

Should these be 3_0_1 now with OpenSSL 3.0.1?

Asking because I believe curl is comparing header version with library versions via a call to SSL_CTX_load_verify_dir which finds the library version here.
@mattcaswell
Copy link
Member

Should these be 3_0_1 now with OpenSSL 3.0.1?

No. They need to remain stable across patch/minor change bumps. If we add new symbols then they would get the higher version number

@t8m t8m added resolved: answered The issue contained a question which has been answered triaged: question The issue contains a question labels Jan 14, 2022
@mattcaswell mattcaswell mentioned this issue Jan 14, 2022
Closed
@satmandu satmandu mentioned this issue Jan 14, 2022
Closed
@tmshort tmshort closed this as completed Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
resolved: answered The issue contained a question which has been answered triaged: question The issue contains a question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@satmandu @t8m @mattcaswell @tmshort