-
Notifications
You must be signed in to change notification settings - Fork 24.4k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Update RNTester documentation for Mac M1
Summary: This Diff updates the RNTester README to support Mac M1 laptops. ## Changelog [General][Changed] - Updated `rn-tester/README.md` to include instructions for Mac M1 users Reviewed By: cortinico Differential Revision: D34376951 fbshipit-source-id: 7195bddefb6e202261fef9b5f5225484c506fd9b
- Loading branch information
1 parent
200488e
commit 4ec2d6c
Showing
1 changed file
with
10 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4ec2d6c
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.
Hi,
I just tried this and I got an error running the pod install command
Running the pod install command without the emulation worked.
I also noticed that I have built previous apps with the default commands and didn't encounter any issues.
4ec2d6c
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.
Maybe @cipolleschi can help you over here?
4ec2d6c
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.
Hi @Raraku, thanks for reporting this.
To help you out, I could need few more information:
ffi
witharch -x86_64
?bundle exec pod install
orbundle exec arch -x86_64 pod install
?which ruby
version have you installed/are you running?4ec2d6c
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.
Hi @cipolleschi
sudo arch -x86_64 gem install ffi
.arch -x86_64 pod install
ruby -v
I got version 2.6.8.Please note that just running pod install worked and I was able to build the application
4ec2d6c
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 run a few tests here and there to try to understand what's going on. I managed to reproduce your issue :D
First of all, that's not an error but a warning. If you open the
RNTesterPods.xcworspace
you should be able to build and run the app without issues. That's a good news.The warning suggests to run this command
env /usr/bin/arch -arm64 /bin/bash --login
which logs you into abash
shell. However, this does not change a thing.Looking into similar issues, I found this other related issue. This suggests to install cocoapods using brew instead of using bundler. But at this point, we can't use the
bundle exec
version of the command anymore, but we have to use thearch -x86_64 pod install
directly.If we do that, the warnings go away. If we reinstall
cocoapods
from gem, it comes back.At the moment, I'm using gem 3.0.3.1, from
gem -v
.The note was introduced to fix this error
I can't reproduce this error at the moment.
My best guess at the moment is that something has been updated since the 18th of Feb, when I incurred in the issue. Bundler has been updated the 23rd and it may now install a proper version.
@cortinico Maybe we should update the doc linking these resources and specifying that the issue may be caused by older versions of
ruby
,gem
andbundler
? We can suggest first to update the environment (the best practice to handle ruby is to use a ruby version manager likerbenv
and, if the problem persist, to manually installffi
andcocoapods
with the arch flag. What do you think?@Raraku if it works for you with the simple
pod install
you should be able to work with that without issues, though. Thanks for reporting this problem.4ec2d6c
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 guess my point is the same as the one I wrote here: facebook/react-native-website#3003
Let's find the most universal & easiest command we can share and let's update the docs with that one. I don't have an M1 so I can't really try anything at this stage