Skip to content

Cocoapods ffi error can be in stdout or stderr - #121470

Merged
auto-submit[bot] merged 4 commits into
flutter:masterfrom
tvolkert:ffi
Feb 28, 2023
Merged

Cocoapods ffi error can be in stdout or stderr#121470
auto-submit[bot] merged 4 commits into
flutter:masterfrom
tvolkert:ffi

Conversation

@tvolkert

@tvolkerttvolkert commented Feb 26, 2023•
edited
Loading

Copy link
Copy Markdown
Contributor

I was seeing the following error show up in stdout:

LoadError - dlopen(/Library/Ruby/Gems/2.6.0/gems/ffi-1.15.5/lib/ffi_c.bundle, 0x0009)
...

And it was solved by running:

sudo gem uninstall ffi && sudo gem install ffi -- --enable-libffi-alloc

#99195

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], including [Features we expect every widget to implement].
  • I signed the [CLA].
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is [test-exempt].
  • All existing and new tests are passing.

@flutter-dashboardflutter-dashboardBot added the tool Affects the "flutter" command-line tool. See also t: labels. label Feb 26, 2023
jmagman
jmagman reviewed Feb 27, 2023
View reviewed changes

@jmagmanjmagman 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.

x-ref #102327

Comment threadpackages/flutter_tools/lib/src/macos/cocoapods.dart Outdated
christopherfujino
christopherfujino reviewed Feb 27, 2023
View reviewed changes
Comment threadpackages/flutter_tools/test/general.shard/macos/cocoapods_test.dart Outdated
tvolkertand others added 4 commits February 27, 2023 13:20
@tvolkert
Cocoapods ffi error can be in stdout or stderr
516db74
I was seeing the following error show up in _stdout_:
```
LoadError - dlopen(/Library/Ruby/Gems/2.6.0/gems/ffi-1.15.5/lib/ffi_c.bundle, 0x0009)
...
```
And it was solved by running:
```sh
sudo gem uninstall ffi && sudo gem install ffi -- --enable-libffi-alloc
```
@tvolkert@christopherfujino
Update packages/flutter_tools/test/general.shard/macos/cocoapods_test…
5b77fad
….dart
Co-authored-by: Christopher Fujino <fujino@google.com>
@tvolkert@jmagman
Update packages/flutter_tools/lib/src/macos/cocoapods.dart
1d8c642
Co-authored-by: Jenn Magder <magder@google.com>
@tvolkert
Apply rest of suggested change
f4c875b
christopherfujino
christopherfujino approved these changes Feb 27, 2023
View reviewed changes

@christopherfujinochristopherfujino left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@tvolkerttvolkert added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 27, 2023
jmagman
jmagman approved these changes Feb 27, 2023
View reviewed changes

@jmagmanjmagman 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 thanks @tvolkert!

@auto-submit
auto-submitBot merged commit c4ef072 into flutter:masterFeb 28, 2023
@engine-flutter-autorollengine-flutter-autoroll mentioned this pull request Feb 28, 2023
Merged
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 28, 2023
@engine-flutter-autoroll
c4ef072Cocoapods ffi error can be in stdout or stderr (flutter/flutt…
ee0cef7
…er#121470)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 10, 2023
@engine-flutter-autoroll
c4ef072Cocoapods ffi error can be in stdout or stderr (flutter/flutt…
b1426b4
…er#121470)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 10, 2023
@engine-flutter-autoroll
c4ef072Cocoapods ffi error can be in stdout or stderr (flutter/flutt…
f98c563
…er#121470)
@tvolkert
tvolkert deleted the ffi branch October 23, 2023 02:03
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jmagmanjmagmanjmagman approved these changes
+1 more reviewer
@christopherfujinochristopherfujinochristopherfujino approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

autosubmitMerge PR when tree becomes green via auto submit ApptoolAffects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tvolkert@jmagman@christopherfujino