Skip to content

cmake/FindCares: fix version detection for c-ares 1.34.1 - #15368

Closed
vszakats wants to merge 1 commit into
curl:masterfrom
vszakats:cm-ares-vers
Closed

cmake/FindCares: fix version detection for c-ares 1.34.1#15368
vszakats wants to merge 1 commit into
curl:masterfrom
vszakats:cm-ares-vers

Conversation

@vszakats

Copy link
Copy Markdown
Member

Due to a regression in c-ares 1.34.1, the non-pkg-config version
detection method broke for this version. c-ares 1.34.2 fixes it,
but update our detection code anyway to also work with 1.34.1.

Ref:
c-ares/c-ares@126e274
c-ares/c-ares#903

Regression in 1.34.1, fixed in 1.34.2. Make sure we detect version
also for the 1.34.1 version.

c-ares/c-ares@126e274
c-ares/c-ares#903
@vszakats vszakats added the cmake label Oct 22, 2024
@vszakats vszakats changed the title FindCares.cmake: fix raw version detection for c-ares 1.34.1 cmake/FindCares: fix raw version detection for c-ares 1.34.1 Oct 22, 2024
@github-actions github-actions Bot added the build label Oct 22, 2024
@vszakats vszakats changed the title cmake/FindCares: fix raw version detection for c-ares 1.34.1 cmake/FindCares: fix version detection for c-ares 1.34.1 Oct 22, 2024
@vszakats vszakats closed this in 096bfde Oct 22, 2024
@vszakats
vszakats deleted the cm-ares-vers branch October 22, 2024 18:22
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Due to a regression in c-ares 1.34.1, the non-pkg-config version
detection method broke for this version. c-ares 1.34.2 fixes it,
but update our detection code anyway to also work with 1.34.1.

Ref:
c-ares/c-ares@126e274
c-ares/c-ares#903

Closes curl#15368
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant