Skip to content

verify-release: verify more thoroughly with git#22018

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/verify-with-git
Closed

verify-release: verify more thoroughly with git#22018
bagder wants to merge 2 commits into
masterfrom
bagder/verify-with-git

Conversation

@bagder

@bagder bagder commented Jun 15, 2026

Copy link
Copy Markdown
Member

If the script is invoked in a git repository it verifies the tarball better.

@bagder bagder added the script label Jun 15, 2026
@bagder bagder marked this pull request as ready for review June 15, 2026 10:32
@bagder bagder requested a review from Copilot June 15, 2026 10:35

Copilot AI left a comment

Copy link
Copy Markdown

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 enhances scripts/verify-release so that, when run inside an up-to-date curl git checkout, it verifies a release tarball by rebuilding it from the corresponding git tag (instead of only rebuilding from the tarball contents). It also updates the verification documentation accordingly.

Changes:

  • Add git-checkout detection to scripts/verify-release and perform “full verification” by checking out the release tag and running dmaketgz.
  • Keep the previous “lighter verification” path when not in a suitable git checkout.
  • Update docs/VERIFY.md with instructions for running the script from a curl git repository.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
scripts/verify-release Adds git-aware verification flow that checks out a tag and rebuilds the release tarball.
docs/VERIFY.md Documents the new recommended workflow for full verification using a curl git checkout.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/VERIFY.md
Comment thread scripts/verify-release Outdated
If the script is invoked in a git repository it verifies the tarball
better.
@bagder bagder force-pushed the bagder/verify-with-git branch from 1f77596 to 13041d5 Compare June 15, 2026 10:50
Comment thread scripts/verify-release Outdated
@bagder bagder closed this in 6ce7404 Jun 15, 2026
@bagder bagder deleted the bagder/verify-with-git branch June 15, 2026 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants