-
Notifications
You must be signed in to change notification settings - Fork 1.5k
fluidsynth: Update to version 2.5.0, fix checkver & autoupdate #16455
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
base: master
Are you sure you want to change the base?
Conversation
WalkthroughUpdates fluidsynth manifest from version 2.4.8 to 2.5.0, restructuring license field to an object format, replacing bin with env_add_path, updating checkver to GitHub API with jsonpath/regex patterns, and expanding autoupdate with per-architecture URL templates and extract directories. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~40 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
All changes look good. Wait for review from human collaborators. fluidsynth
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
bucket/fluidsynth.json(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: WindowsPowerShell
🔇 Additional comments (3)
bucket/fluidsynth.json (3)
5-8: License field restructuring improves manifest clarity.The migration from a string to a structured object with SPDX identifier and URL is a best-practice improvement that enhances maintainability and provides explicit license information.
27-38: Autoupdate template variables are correctly configured.Verification confirms the template variables properly map to the checkver regex named capture groups:
$matchTagcorrectly maps to(?<tag>v?([\\d.]+))capturing "v2.5.0"$matchPrefixcorrectly maps to(?<prefix>[^\"]+)capturing "fluidsynth-v2.5.0-win10"$matchSuffixcorrectly maps to(?<suffix>-[^\"]+)?capturing "-cpp11"URL template expansion produces the correct asset URLs for both architectures, and extract_dir values use consistent variable substitution. The configuration is valid.
10-19: extract_dir names verified as correct.Both extract_dir values match the actual root folder names inside the ZIPs:
- 64-bit:
fluidsynth-v2.5.0-win10-x64-cpp11/✓- 32-bit:
fluidsynth-v2.5.0-win10-x86-cpp11/✓GitHub API confirmed the release assets exist with the expected names and sizes. Hash verification could not be completed in the sandbox environment due to SSL restrictions, but the file URLs and internal structure are correct. Hash integrity will be validated when users download these files.
Summary
Updates
fluidsynthto version 2.5.0 and restructures the manifest for better maintainability and accuracy.Related issues or pull requests
Changes
32bitarchitecture with explicit extract pathsenv_add_pathinstead of manual PATH modification instructionslicensefield with full SPDX and URLcheckverto use the GitHub API with JSONPath and regex for dynamic asset detectionautoupdatelogic to handle both x64 and x86 builds with flexible matchingNotes
Support for GLib and libinstpatch is hereby deprecated. They will be removed in 2.6.0.
Starting from version 2.6.0, the related assets may revert to names like
fluidsynth-2.x.y-win10-x64.zipandfluidsynth-2.x.y-win10-x86.zip. However, this change will not affect either thecheckverorautoupdatefunctionality.Testing
<manifest-name[@version]|chore>: <general summary of the pull request>Summary by CodeRabbit
Release Notes