Skip to content

[supernovas] Patch for upstream issue #272.#48336

Merged
vicroms merged 1 commit intomicrosoft:masterfrom
attipaci:cmake-export-targets
Nov 18, 2025
Merged

[supernovas] Patch for upstream issue #272.#48336
vicroms merged 1 commit intomicrosoft:masterfrom
attipaci:cmake-export-targets

Conversation

@attipaci
Copy link
Copy Markdown
Contributor

The upstream CMakeLists.txt exported the main library target with a different name than what supernovasConfig.cmake was expecting (see upstream Issue #272). A patch to CMakeLists.txt should fix this issue, until the same fix is released upstream in a couple of months...

  • 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.

@attipaci attipaci marked this pull request as ready for review November 16, 2025 17:40
Copy link
Copy Markdown
Member

@vicroms vicroms left a comment

Choose a reason for hiding this comment

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

Please fix version database. I'm marking this PR as draft, once the changes have been implemented please mark it as Ready for Review.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The contents of this file should be generated by the x-add-version command. Please revert the changes made to this file, then change ports/supernovas/vcpkg.json and increase port-version to 1. Then run vcpkg x-add-version supernovas.

Each PR should only add a new version and must not modify the git-tree of a previously published version.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the help. Did as you told. :-)

@vicroms vicroms marked this pull request as draft November 17, 2025 19:00
@attipaci attipaci force-pushed the cmake-export-targets branch from 4ae39b4 to 181bafe Compare November 18, 2025 12:58
@attipaci attipaci marked this pull request as ready for review November 18, 2025 14:00
@attipaci attipaci requested a review from vicroms November 18, 2025 18:38
@vicroms vicroms merged commit 62bd811 into microsoft:master Nov 18, 2025
18 checks passed
@attipaci attipaci deleted the cmake-export-targets branch November 19, 2025 08:13
luadebug pushed a commit to luadebug/vcpkg that referenced this pull request Nov 21, 2025
geiseri pushed a commit to geiseri/vcpkg that referenced this pull request Dec 18, 2025
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.

2 participants