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

Provide maven repo url override for Robolectric to fix OSS CI #630

Closed
wants to merge 2 commits into from

Conversation

colriot
Copy link
Contributor

@colriot colriot commented Feb 18, 2020

Summary:
On Jan 15, 2020 Maven doesn't support unsecure HTTP communication. More context: https://support.sonatype.com/hc/en-us/articles/360041287334-Central-501-HTTPS-Required
As a consequence this fails the build on CI when Robolectric tries to fetch its deps via 'http://repo1.maven.org/maven2'. This diff uses Robolectric's system prop to override default repo url.

Differential Revision: D19949460

Summary: Abstract away ComponentContext and make all accesses to it explicit by providing a `context` property

fbshipit-source-id: 4a7bf6c8d0d099fb6d36c405b435d8bd50f8d946
Summary:
On Jan 15, 2020 Maven doesn't support unsecure HTTP communication. More context: https://support.sonatype.com/hc/en-us/articles/360041287334-Central-501-HTTPS-Required
As a consequence this fails the build on CI when Robolectric tries to fetch its deps via 'http://repo1.maven.org/maven2'. This diff uses Robolectric's system prop to override default repo url.

Differential Revision: D19949460

fbshipit-source-id: c26ffb0fe54b87d508ceddfe56ff259af4070ac3
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D19949460

@pavlospt
Copy link
Contributor

@colriot is the 1st commit correctly added in this CL 😄 ?

@colriot
Copy link
Contributor Author

colriot commented Feb 18, 2020

@pavlospt nope 🙄

@colriot
Copy link
Contributor Author

colriot commented Apr 28, 2020

Close in favour of #653

@colriot colriot closed this Apr 28, 2020
@colriot colriot deleted the export-D19949460 branch April 28, 2020 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants