Skip to content

build: set soversion to major.minor #1357

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.

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

Merged
merged 1 commit into from
Aug 12, 2022

Conversation

Tachi107
Copy link
Contributor

Release 0.11 broke backwards compatibility, meaning that different cpp-httplib versions are compatible with each other only if the major and minor version numbers are the same.

This patch reflects this in the build systems.

See #1209 for some more context.

@yhirose, is it correct that cpp-httplib only maintains backwards compatibility only if both the major and the minor version components are the same? (i.e. 0.10.3 and 0.10.9 are compatible, but 0.10.9 and 0.11.0 are not)

Cc: @sum01

Release 0.11 broke backwards compatibility, meaning that different
cpp-httplib versions are compatible with each other only if the major
and minor version numbers are the same.

This patch reflects this in the build systems.

See yhirose#1209 for some more context.
@yhirose
Copy link
Owner

yhirose commented Aug 12, 2022

@Tachi107, thank you for the change. Yes, if only the patch version changes, it guarantees the backward compatibility.

@yhirose yhirose merged commit a9cf097 into yhirose:master Aug 12, 2022
@Tachi107 Tachi107 deleted the abi-version-minor branch August 13, 2022 01:55
ExclusiveOrange pushed a commit to ExclusiveOrange/cpp-httplib-exor that referenced this pull request May 2, 2023
Release 0.11 broke backwards compatibility, meaning that different
cpp-httplib versions are compatible with each other only if the major
and minor version numbers are the same.

This patch reflects this in the build systems.

See yhirose#1209 for some more context.
@abouvier
Copy link
Contributor

abouvier commented Jul 8, 2023

The comment is no more correct, it should be:

# Example: if you find_package(httplib 0.5.4)
# then anything >= 0.5.4 and < 0.6 is accepted

@Tachi107
Copy link
Contributor Author

Tachi107 commented Jul 8, 2023

@abouvier what are you referring to?

Edit: got it, feel free to update it, I don't care enough about CMake to do it :)

@yhirose
Copy link
Owner

yhirose commented Jul 8, 2023

@abouvier thanks for the report. Since I don't maintain CMake related files, could you send a pull request as @Tachi107 mentioned? Thanks!

@abouvier abouvier mentioned this pull request Jul 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants