Skip to content

[libevent] Upgrade CMake 3.5#44356

Closed
JonLiu1993 wants to merge 2 commits into
microsoft:masterfrom
JonLiu1993:dev/Jon/libevent
Closed

[libevent] Upgrade CMake 3.5#44356
JonLiu1993 wants to merge 2 commits into
microsoft:masterfrom
JonLiu1993:dev/Jon/libevent

Conversation

@JonLiu1993
Copy link
Copy Markdown
Contributor

Fixes #44336
I have submitted an issue on upstream: libevent/libevent#1782
Failed due to https://cmake.org/cmake/help/latest/release/4.0.html#deprecated-and-removed-features

Compatibility with versions of CMake older than 3.5 has been removed.

  • Changes comply with the maintainer guide.
  • SHA512s are updated for each updated download.
  • The "supports" clause reflects platforms that may be fixed by this new version.
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

@JonLiu1993 JonLiu1993 added category:port-bug The issue is with a library, which is something the port should already support info:internal labels Mar 13, 2025
@Neumann-A
Copy link
Copy Markdown
Contributor

Consider fixing the regressions in #44273 instead so we can simply move everything to support CMake 4

@PhoebeHui PhoebeHui requested a review from Copilot March 28, 2025 06:51
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (4)
  • ports/libevent/portfile.cmake: Language not supported
  • ports/libevent/vcpkg.json: Language not supported
  • versions/baseline.json: Language not supported
  • versions/l-/libevent.json: Language not supported

@maflcko
Copy link
Copy Markdown

maflcko commented Apr 1, 2025

Consider fixing the regressions in #44273 instead so we can simply move everything to support CMake 4

My understanding is that this pull request here would also add support for CMake 4.0, but I haven't tested this myself. (#44273 looks like it should be compatible with the changes here, and also looks still in progress/discussion, so it may be better to move forward here first?)

@JonLiu1993
Copy link
Copy Markdown
Contributor Author

Replaced by PR #44712

@JonLiu1993 JonLiu1993 closed this Apr 3, 2025
@JonLiu1993 JonLiu1993 deleted the dev/Jon/libevent branch April 3, 2025 02:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[arrow] build failure

5 participants