Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit 2670315

Browse files
committed
[iOS] Rename Flutter.dSYM to Flutter.framework.dSYM
Renames our Flutter framework dSYM to `Flutter.framework.dSYM` for consistency with all other dSYM bundle names. In iOS release archives, all other dSYM files are: * `App.framework`: `App.framework.dSYM` * `Runner.app`: `Runner.app.dSYM` We continue to archive the dSYM to `Flutter.dSYM.zip` for backward compatibility with the existing instructions for manual symbolification in `docs/Crashes.md` and to remain compatible with dart-lang/dart-ci's symbolizer which expects `Flutter.dSYM` in [`Symbolizer._symbolizeIosFrames`][symbolizer]. Issue: flutter/flutter#116493 symbolizer: https://github.com/dart-lang/dart_ci/blob/e9fd9884a2c76184f4d20eb8a1cb48f92ec63ab5/github-label-notifier/symbolizer/lib/symbolizer.dart#L530
1 parent 387f6f3 commit 2670315

File tree

1 file changed

+19
-4
lines changed

1 file changed

+19
-4
lines changed

sky/tools/create_full_ios_framework.py

Lines changed: 19 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -147,8 +147,8 @@ def create_framework( # pylint: disable=too-many-arguments
147147
framework_dsym = None
148148
simulator_dsym = None
149149
if args.dsym:
150-
framework_dsym = os.path.splitext(framework)[0] + '.dSYM'
151-
simulator_dsym = os.path.splitext(simulator_framework)[0] + '.dSYM'
150+
framework_dsym = framework + '.dSYM'
151+
simulator_dsym = simulator_framework + '.dSYM'
152152

153153
# Emit the framework for physical devices.
154154
shutil.rmtree(framework, True)
@@ -218,10 +218,25 @@ def zip_archive(dst):
218218
'extension_safe/Flutter.xcframework',
219219
],
220220
cwd=dst)
221-
if os.path.exists(os.path.join(dst, 'Flutter.dSYM')):
221+
222+
# Generate Flutter.dSYM.zip for manual symbolification.
223+
#
224+
# Historically, the framework dSYM was named Flutter.dSYM, so in order to
225+
# remain backward-compatible with existing instructions in docs/Crashes.md
226+
# and existing tooling such as dart-lang/dart_ci, we rename back to that name
227+
#
228+
# TODO(cbracken): remove these archives and the upload steps once we bundle
229+
# dSYMs in app archives. https://github.com/flutter/flutter/issues/116493
230+
framework_dsym = os.path.join(dst, 'Flutter.framework.dSYM')
231+
if os.path.exists(framework_dsym):
232+
renamed_dsym = framework_dsym.replace('Flutter.framework.dSYM', 'Flutter.dSYM')
233+
os.rename(framework_dsym, renamed_dsym)
222234
subprocess.check_call(['zip', '-r', 'Flutter.dSYM.zip', 'Flutter.dSYM'], cwd=dst)
223235

224-
if os.path.exists(os.path.join(dst, 'extension_safe', 'Flutter.dSYM')):
236+
extension_safe_dsym = os.path.join(dst, 'extension_safe', 'Flutter.framework.dSYM')
237+
if os.path.exists(extension_safe_dsym):
238+
renamed_dsym = extension_safe_dsym.replace('Flutter.framework.dSYM', 'Flutter.dSYM')
239+
os.rename(extension_safe_dsym, renamed_dsym)
225240
subprocess.check_call(['zip', '-r', 'extension_safe_Flutter.dSYM.zip', 'Flutter.dSYM'], cwd=dst)
226241

227242

0 commit comments

Comments
 (0)