fix(v3): update macOS minimum deployment target to 12.0 - #5223
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (26)
WalkthroughThis PR updates the macOS minimum system version requirement from 10.15 to 12.0 across all build templates, internal toolchain configuration, example projects, and validation tests. This aligns with Go 1.25+ which embeds ChangesmacOS minimum version 10.15 → 12.0
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
⚔️ Resolve merge conflicts
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (2.12.2)level=error msg="[linters_context] typechecking error: pattern ./...: directory prefix . does not contain main module or its selected dependencies" 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 |
|
🤖 PR Triage Review ✅ Accepted Updates macOS minimum deployment target to 12.0 (v3). Modernizes platform support. Platform: macOS (v3) Next Steps: Dispatching for Mac testing. Reviewed by Wails PR Reviewer Bot |
There was a problem hiding this comment.
Pull request overview
Updates the macOS minimum deployment target from 10.15 to 12.0 across v3 build templates and example apps, aligning with Go 1.25's LC_BUILD_VERSION minos 12.0 so macOS shows a clear "requires macOS 12 or later" message instead of silently failing.
Changes:
- Bumped
LSMinimumSystemVersionto12.0.0inInfo.plist/Info.dev.plisttemplates and example apps. - Bumped
CGO_CFLAGS/CGO_LDFLAGS/MACOSX_DEPLOYMENT_TARGETto12.0in darwinTaskfile.ymltemplate and example apps. - Added
TestMacOSMinimumVersiontest verifying the new values in generated assets.
Reviewed changes
Copilot reviewed 26 out of 26 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| v3/internal/commands/updatable_build_assets/darwin/Info.plist.tmpl | Bump LSMinimumSystemVersion to 12.0.0 |
| v3/internal/commands/updatable_build_assets/darwin/Info.dev.plist.tmpl | Bump LSMinimumSystemVersion to 12.0.0 |
| v3/internal/commands/build_assets/darwin/Taskfile.yml | Bump macOS deployment target flags to 12.0 |
| v3/internal/commands/build-assets_test.go | Add test for min macOS version; tidy struct alignment |
| v3/examples/*/build/darwin/Info.plist, Info.dev.plist | Bump LSMinimumSystemVersion in examples |
| v3/examples/*/build/darwin/Taskfile.yml, file-association/build/Taskfile.darwin.yml | Bump macOS deployment target flags in examples |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Follow-up to #5233, which updated the build-asset templates and darwin Taskfile but not the per-example build/darwin/ files. Updates LSMinimumSystemVersion in Info.plist and Info.dev.plist, and mmacosx-version-min / MACOSX_DEPLOYMENT_TARGET in each example's darwin Taskfile.yml across 22 example projects so they match the new project default. Refs #5161
c311755 to
6af0c80
Compare
|
Self-review note: |
|
Thanks for the overview. No actionable issues identified — the change is straightforward (10.15 → 12.0 minimum deployment target across build templates and example configs). Dropping Catalina/Big Sur support aligns with the v3 beta scope. |
Summary
Updates the macOS minimum deployment target from 10.15 (Catalina) to 12.0 (Monterey) across all build templates and examples.
Go 1.25+ embeds
LC_BUILD_VERSIONwithminos 12.0into Mach-O binaries, making them physically unable to run on macOS < 12. The oldInfo.plistdeclared 10.15 as the minimum, causing macOS to allow the launch but fail silently at the kernel level with no error dialog. SettingLSMinimumSystemVersionto 12.0 ensures macOS displays a clear "requires macOS 12 or later" message on unsupported systems.Changes
v3/internal/commands/updatable_build_assets/darwin/Info.plist.tmpl—10.15.0→12.0.0v3/internal/commands/updatable_build_assets/darwin/Info.dev.plist.tmpl—10.15.0→12.0.0v3/internal/commands/build_assets/darwin/Taskfile.yml—CGO_CFLAGS,CGO_LDFLAGS,MACOSX_DEPLOYMENT_TARGET→12.0v3/examples/*/build/darwin/— same updatesTest
TestMacOSMinimumVersionthat generates build assets and verifiesLSMinimumSystemVersionin bothInfo.plistandInfo.dev.plist, and the deployment target in the darwinTaskfile.yml.Fixes #5161
Summary by CodeRabbit