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

Use correct OpenSSL arch in UWP builds. #24352

Merged
merged 1 commit into from Oct 3, 2019
Merged

Use correct OpenSSL arch in UWP builds. #24352

merged 1 commit into from Oct 3, 2019

Conversation

@jdm
Copy link
Member

jdm commented Oct 3, 2019

This is a followup from #24257. I forgot to run the UWP WACK tests after making my changes, or I would have noticed that the wrong DLLs were being selected for UWP builds.


This change is Reviewable

@highfive
Copy link

highfive commented Oct 3, 2019

Heads up! This PR modifies the following files:

@jdm
Copy link
Member Author

jdm commented Oct 3, 2019

@highfive highfive assigned paulrouget and unassigned SimonSapin Oct 3, 2019
@paulrouget
Copy link
Contributor

paulrouget commented Oct 3, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2019

📌 Commit 6535899 has been approved by paulrouget

@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2019

Testing commit 6535899 with merge d9cbfd2...

bors-servo added a commit that referenced this pull request Oct 3, 2019
Use correct OpenSSL arch in UWP builds.

This is a followup from #24257. I forgot to run the UWP WACK tests after making my changes, or I would have noticed that the wrong DLLs were being selected for UWP builds.

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

bors-servo commented Oct 3, 2019

💔 Test failed - linux-rel-wpt

@CYBAI
Copy link
Collaborator

CYBAI commented Oct 3, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2019

💣 Failed to start rebuilding: Unknown error

@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2019

Testing commit 6535899 with merge e214a8d...

bors-servo added a commit that referenced this pull request Oct 3, 2019
Use correct OpenSSL arch in UWP builds.

This is a followup from #24257. I forgot to run the UWP WACK tests after making my changes, or I would have noticed that the wrong DLLs were being selected for UWP builds.

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

bors-servo commented Oct 3, 2019

💔 Test failed - linux-rel-css

@jdm
Copy link
Member Author

jdm commented Oct 3, 2019

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2019

Testing commit 6535899 with merge d740512...

bors-servo added a commit that referenced this pull request Oct 3, 2019
Use correct OpenSSL arch in UWP builds.

This is a followup from #24257. I forgot to run the UWP WACK tests after making my changes, or I would have noticed that the wrong DLLs were being selected for UWP builds.

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

bors-servo commented Oct 3, 2019

💔 Test failed - linux-rel-css

@jdm
Copy link
Member Author

jdm commented Oct 3, 2019

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2019

Testing commit 6535899 with merge bfc4172...

bors-servo added a commit that referenced this pull request Oct 3, 2019
Use correct OpenSSL arch in UWP builds.

This is a followup from #24257. I forgot to run the UWP WACK tests after making my changes, or I would have noticed that the wrong DLLs were being selected for UWP builds.

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

bors-servo commented Oct 3, 2019

💔 Test failed - status-taskcluster

@jdm
Copy link
Member Author

jdm commented Oct 3, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2019

💣 Failed to start rebuilding: Unknown error

@bors-servo
Copy link
Contributor

bors-servo commented Oct 3, 2019

Testing commit 6535899 with merge c205243...

bors-servo added a commit that referenced this pull request Oct 3, 2019
Use correct OpenSSL arch in UWP builds.

This is a followup from #24257. I forgot to run the UWP WACK tests after making my changes, or I would have noticed that the wrong DLLs were being selected for UWP builds.

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

bors-servo commented Oct 3, 2019

☀️ Test successful - linux-rel-css, linux-rel-wpt, status-taskcluster
Approved by: paulrouget
Pushing c205243 to master...

@bors-servo bors-servo merged commit 6535899 into master Oct 3, 2019
3 checks passed
3 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the jdm-patch-28 branch Dec 12, 2019
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

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