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

Conversation

godofredoc
Copy link
Contributor

@godofredoc godofredoc commented Jun 3, 2023

Use the right binary architectures on scripts running dsymutil.

This is to fix the problem of bad architecture binary for builds running on dart internal: flutter/flutter#128098

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@godofredoc godofredoc marked this pull request as draft June 3, 2023 00:44
@godofredoc godofredoc marked this pull request as ready for review June 5, 2023 21:26
@godofredoc godofredoc changed the title Test arm64 failures. Use the arm64 dsymutil on Mac M1 machines Jun 5, 2023
@godofredoc godofredoc added the autosubmit Merge PR when tree becomes green via auto submit App label Jun 5, 2023
@auto-submit auto-submit bot merged commit 51928f2 into flutter:main Jun 5, 2023
@godofredoc godofredoc deleted the test_clang_arm64 branch June 5, 2023 23:12
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 5, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 6, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 6, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jun 6, 2023
…128301)

flutter/engine@2b353ae...aaa7574

2023-06-06 skia-flutter-autoroll@skia.org Roll Skia from 7db361be474b to e8a71c76b88f (4 revisions) (flutter/engine#42582)
2023-06-05 goderbauer@google.com Remove onSemanticsAction (flutter/engine#42579)
2023-06-05 godofredoc@google.com Use the arm64 dsymutil on Mac M1 machines (flutter/engine#42533)
2023-06-05 skia-flutter-autoroll@skia.org Roll Skia from a226cfbd6cc7 to 7db361be474b (14 revisions) (flutter/engine#42578)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jonahwilliams@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
XilaiZhang pushed a commit to XilaiZhang/engine that referenced this pull request Jun 20, 2023
Use the right binary architectures on scripts running dsymutil.

This is to fix the problem of bad architecture binary for builds running on dart internal: flutter/flutter#128098

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
XilaiZhang added a commit that referenced this pull request Jun 20, 2023
Cherry pick into beta branch
[example build failures on
beta](https://luci-milo.appspot.com/ui/p/dart-internal/builders/flutter/Mac%20engine_release_builder/232/overview)

Use the right binary architectures on scripts running dsymutil.

This is to fix the problem of bad architecture binary for builds running
on dart internal: flutter/flutter#128098

Co-authored-by: godofredoc <godofredoc@google.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants