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

[WIP] Use MSVC linker #24017

Closed
wants to merge 3 commits into from
Closed

[WIP] Use MSVC linker #24017

wants to merge 3 commits into from

Conversation

@jdm
Copy link
Member

jdm commented Aug 20, 2019

This change is Reviewable

@jdm
Copy link
Member Author

jdm commented Aug 20, 2019

@bors-servo try=windows

@bors-servo
Copy link
Contributor

bors-servo commented Aug 20, 2019

Trying commit 890f6c6 with merge 6c48f35...

bors-servo added a commit that referenced this pull request Aug 20, 2019
[WIP] Use MSVC linker

<!-- 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/24017)
<!-- Reviewable:end -->
@jdm jdm removed the S-awaiting-review label Aug 20, 2019
@bors-servo
Copy link
Contributor

bors-servo commented Aug 20, 2019

☀️ Test successful - status-taskcluster
State: approved= try=True

@jdm
Copy link
Member Author

jdm commented Aug 20, 2019

@bors-servo try=windows

@bors-servo
Copy link
Contributor

bors-servo commented Aug 20, 2019

Trying commit df85d20 with merge 0b262fb...

bors-servo added a commit that referenced this pull request Aug 20, 2019
[WIP] Use MSVC linker

<!-- 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/24017)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 20, 2019

☀️ Test successful - status-taskcluster
State: approved= try=True

@jdm jdm force-pushed the jdm-patch-51 branch from df85d20 to 0bb52a4 Aug 21, 2019
@jdm
Copy link
Member Author

jdm commented Aug 21, 2019

@bors-servo try=windows

@bors-servo
Copy link
Contributor

bors-servo commented Aug 21, 2019

Trying commit 0bb52a4 with merge 8a0ebb1...

bors-servo added a commit that referenced this pull request Aug 21, 2019
[WIP] Use MSVC linker

<!-- 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/24017)
<!-- Reviewable:end -->
jdm added 3 commits Aug 20, 2019
@jdm jdm force-pushed the jdm-patch-51 branch from 0bb52a4 to a31e97c Aug 21, 2019
@jdm
Copy link
Member Author

jdm commented Aug 21, 2019

@bors-servo try=windows

@bors-servo
Copy link
Contributor

bors-servo commented Aug 21, 2019

Trying commit a31e97c with merge 208c6ae...

bors-servo added a commit that referenced this pull request Aug 21, 2019
[WIP] Use MSVC linker

<!-- 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/24017)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 21, 2019

💔 Test failed - status-taskcluster

@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2019

The latest upstream changes (presumably #23984) made this pull request unmergeable. Please resolve the merge conflicts.

@jdm jdm closed this Aug 30, 2019
@jdm jdm deleted the jdm-patch-51 branch Aug 30, 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

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