-
Notifications
You must be signed in to change notification settings - Fork 29.7k
macOS: Copy macOS framework dSYM into build outputs #153975
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
Conversation
643924e to
c415a44
Compare
packages/flutter_tools/test/general.shard/build_system/targets/macos_test.dart
Show resolved
Hide resolved
4f48183 to
31f868c
Compare
| await super.build(environment); | ||
|
|
||
| // Copy Flutter framework dSYM (debug symbol) bundle, if present. | ||
| final String? buildModeEnvironment = environment.defines[kBuildMode]; |
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.
I'm guessing yes because it's not being copied into the app, but this is safe to do post-codesigning the framework, right?
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.
Yep -- this is just copying the already signed-by-us dSYM as a sibling into the same output directory, not embedding in any container. Both the app bundle and the App dSYM later get embedded into the xcarchive, but that's a separate step currently handled by Xcode. As is, those archives pass app validation in manual testing.
d68bc17 to
296994b
Compare
As of Xcode 16, App Store validation now requires that apps uploaded to the App store bundle dSYM debug information bundles for each Framework they embed. dSYM bundles are packaged in the FlutterMacOS.xcframework shipped in the `darwin-x64-release` tools archive as of engine patches: * flutter/engine#54696 This copies the FlutterMacOS.framework.dSYM bundle from the tools cache to the build outputs produced by `flutter build macos`. Issue: https://github.com/flutter/flutter/issue/153879
Also use const to do concatenation at compile time.
0bd8dc9 to
942d191
Compare
942d191 to
85f1d43
Compare
|
auto label is removed for flutter/flutter/153975, due to - The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label. |
|
Verified presubmits on cl/666904278 and they had passed. |
As of Xcode 16, App Store validation now requires that apps uploaded to the App store bundle dSYM debug information bundles for each Framework they embed. dSYM bundles are packaged in the FlutterMacOS.xcframework shipped in the `darwin-x64-release` tools archive as of engine patches: * flutter/engine#54696 This copies the FlutterMacOS.framework.dSYM bundle from the tools cache to the build outputs produced by `flutter build macOS`. Fixes: flutter#153879
As of Xcode 16, App Store validation now requires that apps uploaded to the App store bundle dSYM debug information bundles for each Framework they embed.
dSYM bundles are packaged in the FlutterMacOS.xcframework shipped in the
darwin-x64-releasetools archive as of engine patches:This copies the FlutterMacOS.framework.dSYM bundle from the tools cache to the build outputs produced by
flutter build macOS.Fixes: #153879
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.