Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update SUVersionDisplay to allow including build versions and update information #2321

Merged
merged 5 commits into from
Feb 25, 2023

Conversation

zorgiepoo
Copy link
Member

@zorgiepoo zorgiepoo commented Feb 20, 2023

This deprecates the existing format method on SUVersionDisplay and adds two new methods to the protocol which allows the formatter to know about the build & display versions and the update. One of these methods is used for formatting both the update and bundle versions, while the other (optional) method is used for formatting only the bundle version.

Fixes #2267

Misc Checklist

  • My change requires a documentation update on Sparkle's website repository
  • My change requires changes to generate_appcast, generate_keys, or sign_update

Only bug fixes to regressions or security fixes are being backported to the 1.x (master) branch now. If you believe your change is significant enough to backport, please also create a separate pull request against the master branch.

Testing

I tested and verified my change by using one or multiple of these methods:

  • Sparkle Test App
  • Unit Tests
  • My own app
  • Other (please specify)

Tested default and custom format strings provided by a user driver delegate when:

  • A new update is available (formats both update and bundle display versions)
  • No new update is available and the user is on the latest known update in the feed (formats bundle display version)
  • No new update is available but the user is on an update newer than latest one known in the feed (formats both update and bundle display versions)
  • No new update is available because the user cannot install it (e.g. due to min OS requirement) (formats both update and bundle display versions, but the bundle display version is not displayed, however it's inferred to differ for comparison)

macOS version tested: 13.0.1 (22A400)

@zorgiepoo zorgiepoo merged commit 87697ce into 2.x Feb 25, 2023
@zorgiepoo zorgiepoo deleted the update-version-display branch February 25, 2023 16:15
danielpunkass added a commit to danielpunkass/Sparkle that referenced this pull request Feb 27, 2023
…s on my branch.

* commit '362f50a8a19cfb1a18202c5912ce4ce6f94d0190': (60 commits)
  Exit with an error if generate_appcast cannot sign an update (sparkle-project#2322)
  Update SUVersionDisplay to allow including build versions and update information (sparkle-project#2321)
  Update Package management files for version 2.3.2
  Update CHANGELOG for 2.3.2
  Remove unnecessary min macro checks (sparkle-project#2318)
  Add support for plain text release notes view (sparkle-project#2315)
  Don't construct appcast item if enclosure URL is invalid (sparkle-project#2317)
  Finnish localization grammar fixes (sparkle-project#2311)
  Reduce code size and make codebase more consistent (sparkle-project#2305)
  Add delegate method to hide showing version history option (sparkle-project#2303)
  Update Package management files for version 2.3.1
  Update CHANGELOG for 2.3.1
  Add deprecations/warnings for incorrect feed & update checking usage (sparkle-project#2295)
  Log the URL that failed to download correctly (sparkle-project#2296)
  Remove duplicate Sparkle localization strings in Updater app (sparkle-project#2288)
  Ask permission for automatically downloading and installing new updates (sparkle-project#2285)
  Upgrade CI to default to using Xcode 14.1 (sparkle-project#2287)
  Enable deployment post processing for Release configurations (sparkle-project#2286)
  Fix update permission alert title text overlapping with question text (sparkle-project#2284)
  Update Czech translation (sparkle-project#2275)
  ...
@zorgiepoo zorgiepoo added this to the 2.4 milestone Mar 4, 2023
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.

Add ability to show both CFBundleVersion and CFBundleShortVersionString in UI
1 participant