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

Fix SSL errors on travis CI. #1697

Merged
merged 1 commit into from Sep 13, 2017
Merged

Fix SSL errors on travis CI. #1697

merged 1 commit into from Sep 13, 2017

Conversation

@glennw
Copy link
Member

glennw commented Sep 12, 2017

This change is Reviewable

@glennw
Copy link
Member Author

glennw commented Sep 12, 2017

Not for merging - just testing right now.

@glennw glennw force-pushed the glennw:ssl-fix branch 13 times, most recently from 41bd463 to 045ba48 Sep 12, 2017
@glennw glennw changed the title [WIP] Test if this helps with the SSL errors we are seeing. Fix SSL errors on travis CI. Sep 13, 2017
@glennw
Copy link
Member Author

glennw commented Sep 13, 2017

@metajack
Copy link

metajack commented Sep 13, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Sep 13, 2017

📌 Commit 045ba48 has been approved by metajack

@bors-servo
Copy link
Contributor

bors-servo commented Sep 13, 2017

Testing commit 045ba48 with merge 2c6e38b...

bors-servo added a commit that referenced this pull request Sep 13, 2017
Fix SSL errors on travis CI.

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/1697)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 13, 2017

💔 Test failed - status-travis

@glennw
Copy link
Member Author

glennw commented Sep 13, 2017

@bors-servo retry

  • seemingly unrelated error (memory?)
@bors-servo
Copy link
Contributor

bors-servo commented Sep 13, 2017

Testing commit 045ba48 with merge 4293ea6...

bors-servo added a commit that referenced this pull request Sep 13, 2017
Fix SSL errors on travis CI.

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/1697)
<!-- Reviewable:end -->
@glennw glennw force-pushed the glennw:ssl-fix branch from 045ba48 to 87f41a0 Sep 13, 2017
@glennw
Copy link
Member Author

glennw commented Sep 13, 2017

@bors-servo r=metajack

Also includes an update to osmesa-src which seems to fix an inexplicably new c++ compiler error. The osmesa version also matches what we are using in Servo now, which is a good thing anyway.

@bors-servo
Copy link
Contributor

bors-servo commented Sep 13, 2017

📌 Commit 87f41a0 has been approved by metajack

@bors-servo
Copy link
Contributor

bors-servo commented Sep 13, 2017

Testing commit 87f41a0 with merge 8e147bd...

bors-servo added a commit that referenced this pull request Sep 13, 2017
Fix SSL errors on travis CI.

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/1697)
<!-- Reviewable:end -->
@glennw glennw force-pushed the glennw:ssl-fix branch from 87f41a0 to 2f2f112 Sep 13, 2017
Update to python 2.7.13 which fixes the SSL error. Also install
in a virtualenv due to permissions problems when installing
MarkupSafe.
@glennw glennw force-pushed the glennw:ssl-fix branch from 2f2f112 to 3670d23 Sep 13, 2017
@glennw
Copy link
Member Author

glennw commented Sep 13, 2017

@bors-servo r=metajack p=1000

@bors-servo
Copy link
Contributor

bors-servo commented Sep 13, 2017

📌 Commit 3670d23 has been approved by metajack

bors-servo added a commit that referenced this pull request Sep 13, 2017
Fix SSL errors on travis CI.

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/1697)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 13, 2017

Testing commit 3670d23 with merge a51069c...

@bors-servo
Copy link
Contributor

bors-servo commented Sep 13, 2017

☀️ Test successful - status-travis
Approved by: metajack
Pushing a51069c to master...

@bors-servo bors-servo merged commit 3670d23 into servo:master Sep 13, 2017
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@@ -655,8 +655,8 @@ dependencies = [

[[package]]
name = "osmesa-src"
version = "12.0.1"
source = "git+https://github.com/servo/osmesa-src#42509fbd70d0e54e1278849b771ea1cb1c3386f4"
version = "17.2.0-devel"

This comment has been minimized.

@kvark

kvark Sep 13, 2017

Member

oh wow, I wonder if we can use LLVM4 now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.