Skip to content

build(samples): Generate Xcode metadata - #8982

Merged
philprime merged 5 commits into
mainfrom
philprime/upload-dsym-scripts
Sep 9, 2026
Merged

build(samples): Generate Xcode metadata#8982
philprime merged 5 commits into
mainfrom
philprime/upload-dsym-scripts

Conversation

@philprime

@philprime philprime commented Sep 8, 2026

Copy link
Copy Markdown
Member

Move sample Info.plist and entitlements definitions into XcodeGen so the YAML specs are the single source of truth. Generated metadata files are now ignored, while applicable app targets inject git information and upload debug symbols during builds.

Verification

  • make format
  • make xcode-ci
  • make analyze
  • Built isolated iOS, macOS, tvOS, visionOS, and watchOS samples
  • Confirmed generated metadata files are recreated and ignored

The workspace-based iOS-Swift build was also attempted, but remains blocked by the existing duplicate-header collision in shared DerivedData.

#skip-changelog

Closes #8983

Move sample Info.plist and entitlements definitions into XcodeGen and ignore the generated files. Add git metadata and debug-symbol upload phases to applicable sample app targets.
@philprime
philprime marked this pull request as ready for review September 8, 2026 13:29
@philprime philprime self-assigned this Sep 8, 2026
@philprime philprime added the ready-to-merge (deprecated) Use run-full-ci instead. label Sep 8, 2026
@philprime philprime added run-full-ci Allows gated GitHub Action workflows to run for a labelled pull request and removed ready-to-merge (deprecated) Use run-full-ci instead. labels Sep 8, 2026
Comment thread Samples/watchOS-Swift/watchOS-Swift.yml
@sentry

sentry Bot commented Sep 8, 2026

Copy link
Copy Markdown

📲 Install Builds

iOS

🔗 App Name App ID Version Configuration
SDK-Size io.sentry.sample.SDK-Size 9.27.0 (1) Release

⚙️ sentry-cocoa Build Distribution Settings

Comment thread Samples/macOS-Swift/macOS-Swift.yml

@NinjaLikesCheez NinjaLikesCheez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but one comment for your consideration

Comment thread Samples/DistributionSample/DistributionSample.yml Outdated
@philprime
philprime enabled auto-merge (squash) September 9, 2026 09:47
Comment thread Samples/iOS-Swift/iOS-Swift.yml
@philprime
philprime merged commit d0d338a into main Sep 9, 2026
191 of 192 checks passed
@philprime
philprime deleted the philprime/upload-dsym-scripts branch September 9, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved run-full-ci Allows gated GitHub Action workflows to run for a labelled pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

build(samples): Generate Xcode metadata

2 participants