Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update gradle-download-task from 2.0.0 to 3.1.2 to fix boost zip down… #11298

Closed

Conversation

younthu
Copy link
Contributor

@younthu younthu commented Dec 5, 2016

Small pull request

motivation
The doc 'Building React Native from source' is not up to date now.

While i try to build from the source my self, the gradle-download-task(2.0.0) will always fail to download boost zip with a failure of 403.

After i upgrade the gradle-download-task to 3.1.2, the download works well.

Test plan (required)
I tested to build the source myself, it works.

This is non-code update, it is document update, hope to help other people who get the same download failure.

Code formatting
Non-code update PR.

@facebook-github-bot
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @christopherdro and @mikelambert to be potential reviewers.

@facebook-github-bot
Copy link
Contributor

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks!

If you are contributing on behalf of someone else (eg your employer): the individual CLA is not sufficient - use https://developers.facebook.com/opensource/cla?type=company instead. Contact cla@fb.com if you have any questions.

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Dec 5, 2016
@facebook-github-bot
Copy link
Contributor

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@mkonicek
Copy link
Contributor

mkonicek commented Dec 5, 2016

This is awesome, thanks!

Android tests are currently broken on master but once they're fixed could you rebase this to have green Android tests on the PR?

@younthu
Copy link
Contributor Author

younthu commented Dec 6, 2016

@mkonicek

I updated the PR, all those tests are green now.

Can you help to have a review?

Thanks.

@christopherdro
Copy link
Contributor

@facebook-github-bot shipit

@janicduplessis
Copy link
Contributor

Could you also update https://github.com/facebook/react-native/blob/master/build.gradle#L10. Not sure if it's too late since it was shipped (was 8 hours so it might have failed).

@younthu
Copy link
Contributor Author

younthu commented Dec 7, 2016

I can try it, but not sure it is the right process or not.

@chirag04
Copy link
Contributor

chirag04 commented Dec 8, 2016

@facebook-github-bot shipit

mkonicek pushed a commit that referenced this pull request Dec 12, 2016
Summary:
Small pull request

 **motivation**
The doc 'Building React Native from source' is not up to date now.

While i try to build from the source my self, the gradle-download-task(2.0.0) will always fail to download boost zip with a failure of 403.

After i upgrade the gradle-download-task to 3.1.2, the download works well.

**Test plan (required)**
I tested to build the source myself, it works.

This is non-code update, it is document update, hope to help other people who get the same download failure.

**Code formatting**
Non-code update PR.
Closes #11298

Differential Revision: D4288387

fbshipit-source-id: 5e6feef1ea80f7e3feddcc7e4abf2b5cea82f08f
DanielMSchmidt pushed a commit to DanielMSchmidt/react-native that referenced this pull request Jan 4, 2017
Summary:
Small pull request

 **motivation**
The doc 'Building React Native from source' is not up to date now.

While i try to build from the source my self, the gradle-download-task(2.0.0) will always fail to download boost zip with a failure of 403.

After i upgrade the gradle-download-task to 3.1.2, the download works well.

**Test plan (required)**
I tested to build the source myself, it works.

This is non-code update, it is document update, hope to help other people who get the same download failure.

**Code formatting**
Non-code update PR.
Closes facebook#11298

Differential Revision: D4288387

fbshipit-source-id: 5e6feef1ea80f7e3feddcc7e4abf2b5cea82f08f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants