lately macOS: fix the two build-breakers found in pre-flight review #32
No reviewers
Labels
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
key-store/key.store!32
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "lately-macos-preflight-fixes"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Pre-flight review (rules_apple 4.5.3 source-checked) of the gated
lately_macos_releasetarget found two hard failures before compile: an--embed_labelthat cannot matchapple_release_version'sbuild_label_pattern, and$(AppIdentifierPrefix)in the entitlements of a no-profile build. Also fixes the macOS push entitlement key (com.apple.developer.aps-environment, which is what the synced Developer ID profile grants), matches SwiftPM's macOS 15 floor, and removes the deadwrite_lately_macos_plist.🤖 Generated with Claude Code
Review of the new lately_macos_release target (still gated behind include_lately_macos) against rules_apple 4.5.3 found two hard failures before any compile step, plus a runtime bug and stale helpers: * build_lately_macos passed --embed_label=lately_<v>_build_<n>, but the target's version is :apple_release_version whose build_label_pattern is key.store_{version}_build_{build}; versiontool has no fallback when a label is present and does not match, so the AppleBundleVersion action fails. Use the key.store_ label like every other invocation. * The target has no provisioning_profile (the lane builds ad-hoc and re-signs with Developer ID), but Lately.entitlements references $(AppIdentifierPrefix), which plisttool can only substitute from a profile -- ProcessEntitlementsFiles fails. Point the target at Lately-DeveloperID.entitlements, which hard-codes the team prefix and is exactly what sign-lately-macos-app applies. * Both macOS entitlements files used the iOS key aps-environment; AppKit wants com.apple.developer.aps-environment, which is what the synced MAC_APP_DIRECT profile grants. Without the fix the pipeline passes but remote-notification registration fails at runtime. * minimum_os_version 15.0 to match the .macOS(.v15) floor the SwiftPM package declared (Info.plist LSMinimumSystemVersion likewise). * Drop the dead write_lately_macos_plist helper and its LatelyMac executable name; the Bazel bundle's executable is Lately. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>