Skip to content

Prepare manual vcpkg release updates for minimal SQLite and Android curl - #1545

Open
bmehta001 wants to merge 4 commits into
microsoft:mainfrom
bmehta001:fix/vcpkg-release-features
Open

bmehta001 wants to merge 4 commits into
microsoft:mainfrom
bmehta001:fix/vcpkg-release-features

Conversation

@bmehta001

@bmehta001 bmehta001 commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Why

The SDK overlay already declares minimal-sqlite and android-curl-openssl, but the official vcpkg port does not expose them. A version-only registry bump would leave its manifest and feature wiring behind. Prepare the complete port manually after the next SDK release rather than requiring a personal token for release-triggered PR automation.

Changes

  • Remove the automated vcpkg-release-bump.yml workflow and its PAT/fork configuration requirements.
  • Promote the complete overlay port from the release tag using a manual helper, including its manifest, CMake wiring, and any patches, before updating the release version and archive hash. Remove obsolete port files and reset port-version.
  • Add a token-free preparation helper for the manual vcpkg PR after the next SDK release. Validate the required features, release metadata, and destination before replacing the registry port.
  • Preserve the default consumer graph while adding Linux minimal-SQLite coverage and Android default, minimal-SQLite Java/JNI, minimal-SQLite curl/OpenSSL, and minimal-SQLite curl/mbedTLS build variants.
  • Document current registry feature availability and the manual release promotion procedure in both the vcpkg guide and maintainer onboarding.

This PR does not cut a release, change the currently published registry port, configure a release token, or open a vcpkg PR. Merge it before cutting the next SDK release; the subsequent registry PR can then expose these features with matching released source.

Validation

  • Six release-port regression tests pass on Windows and Linux/WSL, with no skipped or excluded tests.
  • The remaining modified workflow passes actionlint; shell syntax checks and the repository's misspell check pass for the changed documentation and scripts.
  • The consumer manifest is formatted using vcpkg format-manifest.
  • Full Linux/Android port builds are covered by the expanded PR CI matrix; they were not run locally.

Carry feature declarations and build wiring from the release tag instead of only changing the registry version and hash. Keep a token-free helper for manual registry updates and exercise opt-in dependency graphs in CI.

Files changed:
.github/scripts/prepare-vcpkg-release.py
.github/workflows/test-vcpkg.yml
.github/workflows/vcpkg-release-bump.yml
docs/building-with-vcpkg.md
tests/vcpkg/test-release-port.py
tests/vcpkg/test-vcpkg-android.sh
tests/vcpkg/test-vcpkg-linux.sh
tests/vcpkg/vcpkg.json

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bmehta001
bmehta001 requested a review from a team as a code owner October 5, 2026 10:25
@bmehta001 bmehta001 self-assigned this Oct 5, 2026
Keep downstream registry PRs manual so releasing the SDK does not require a maintainer PAT in repository automation. Retain the local port preparation helper and document the complete manual promotion and version-database sequence.

Files changed:
.github/workflows/vcpkg-release-bump.yml (removed)
docs/building-with-vcpkg.md
docs/maintainer-onboarding.md

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: bd3ff552-cbf8-4164-bc50-3c26778f76de
@bmehta001 bmehta001 changed the title Carry minimal SQLite and Android curl features into release port updates Prepare manual vcpkg release updates for minimal SQLite and Android curl Oct 5, 2026
bmehta001 and others added 2 commits October 5, 2026 05:52
Preserve the modules and Collector team additions while distinguishing reviewed submodule pins from moving HEADs. Align both guides with manual port preparation and prevent local overlay tests from being mistaken for published-archive validation.

Files changed:
docs/maintainer-onboarding.md
docs/building-with-vcpkg.md

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: bd3ff552-cbf8-4164-bc50-3c26778f76de

This branch has not been deployed

No deployments
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.

1 participant