Skip to content

Conversation

@alt-graph
Copy link
Member

This replaces the previous wrap file with the latest "official" one from the WrapDB under https://mesonbuild.com/Wrapdb-projects.html.

This replaces the previous wrap file with the latest "official" one from
the WrapDB under https://mesonbuild.com/Wrapdb-projects.html.

Signed-off-by: Lars Froehlich <lars.froehlich@desy.de>
@alt-graph alt-graph self-assigned this Jun 13, 2025
@alt-graph alt-graph added the bug Something isn't working label Jun 13, 2025
Copy link

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.

Pull Request Overview

This PR replaces the Homebrew-based Catch2 wrap with the official WrapDB version (v3.8.1), updating metadata and dependency mappings accordingly.

  • Switched from a [wrap-git] stanza to a [wrap-file] using Catch2 v3.8.1 from WrapDB
  • Updated source_url, source_hash, and added wrapdb_version and fallback URL
  • Adjusted provided dependency names to align with the new wrap
Comments suppressed due to low confidence (3)

subprojects/catch2.wrap:11

  • The key catch2-with-main uses a hyphen, while the other key catch2 does not. For consistency, consider renaming to catch2_with_main.
catch2-with-main = catch2_with_main_dep

subprojects/catch2.wrap:2

  • [nitpick] Consider using a lowercase directory name (e.g., catch2-3.8.1) for consistency with other subproject directories and to avoid potential platform case-sensitivity issues.
directory = Catch2-3.8.1

Copy link
Contributor

@Finii Finii left a comment

Choose a reason for hiding this comment

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

Excellent, thank you!

@alt-graph alt-graph merged commit 6c5f122 into main Jun 16, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants