Skip to content

fix(release): compare signed appcast enclosure#12

Merged
hcaiano merged 1 commit into
mainfrom
fix/release-appcast-enclosure-comparison
Jul 10, 2026
Merged

fix(release): compare signed appcast enclosure#12
hcaiano merged 1 commit into
mainfrom
fix/release-appcast-enclosure-comparison

Conversation

@hcaiano

@hcaiano hcaiano commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Summary

  • compare the full generated and committed Sparkle enclosure lines
  • treat URL-only matches as changed when signature or length differs
  • keep true no-op reruns free of appcast commits

Test plan

  • swift build
  • swift run cycler-tests (ok — 106 checks passed)
  • actionlint .github/workflows/release.yml
  • bash -n Scripts/sparkle-appcast.sh
  • shell assertion: identical enclosure is unchanged and modified length is changed
  • git diff --check

Runtime evidence

  • successful run 29111015018 rebuilt and uploaded a 4177304-byte DMG while the existing appcast still signed 4177027 bytes; URL-only comparison skipped the required appcast update

@hcaiano hcaiano merged commit 72f2ab7 into main Jul 10, 2026
3 checks passed
@hcaiano hcaiano deleted the fix/release-appcast-enclosure-comparison branch July 10, 2026 17:35
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