Skip to content

Conversation

@dulmandakh
Copy link
Contributor

@dulmandakh dulmandakh commented Oct 17, 2020

bump Buck to 2020.09.09.01, and Android command line tools to latest version.

@dulmandakh
Copy link
Contributor Author

@gengjiawen it's ready to merge. Please merge

Copy link
Member

@gengjiawen gengjiawen left a comment

Choose a reason for hiding this comment

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

I don't think we should bundle two ndk until the code ready. This will make the image much bigger.

If you want to debug the new ndk, there will be many ways. I don't this will be a good idea.

@dulmandakh
Copy link
Contributor Author

I'm doing this per discussion on facebook/react-native#29013

@gengjiawen
Copy link
Member

I'm doing this per discussion on facebook/react-native#29013

Still the point stands. If you want to try new ndk, you can use circleCI remote ssh or personal docker image.
But put it in the default image used by many default is a terrible idea.

@dulmandakh
Copy link
Contributor Author

ok, removed NDK 21. Bumping Buck and Android command line tools.

@dulmandakh dulmandakh changed the title install NDK 21 bump Buck to 2020.09.09.01 Oct 18, 2020
@gengjiawen gengjiawen merged commit a647411 into react-native-community:master Oct 18, 2020
@dulmandakh dulmandakh deleted the ndk-21 branch May 27, 2021 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants