Skip to content

Document Windows installation hacks#12578

Merged
bors-servo merged 1 commit intoservo:masterfrom
larsbergstrom:windows_deps
Jul 25, 2016
Merged

Document Windows installation hacks#12578
bors-servo merged 1 commit intoservo:masterfrom
larsbergstrom:windows_deps

Conversation

@larsbergstrom
Copy link
Copy Markdown
Contributor

@larsbergstrom larsbergstrom commented Jul 24, 2016

This is based on the awesome work by @UK992 in #12573.

Basically, it downgrades GCC on Windows to 5.4 within the mingw environment. This is a stopgap until #11756 is completed and we no longer need mingw.

Fixes #12512

r? @metajack


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes fix #__ (github issue number if applicable).
  • There are tests for these changes OR
  • These changes do not require tests because _____

This change is Reviewable

@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jul 24, 2016
@metajack
Copy link
Copy Markdown
Collaborator

@bors-servo r+

Yuck. Do we have any idea why GCC 6 is not working?

@bors-servo
Copy link
Copy Markdown
Contributor

📌 Commit 94c3431 has been approved by metajack

@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-awaiting-review There is new code that needs to be reviewed. labels Jul 25, 2016
@bors-servo
Copy link
Copy Markdown
Contributor

⌛ Testing commit 94c3431 with merge 521f8c0...

bors-servo pushed a commit that referenced this pull request Jul 25, 2016
Document Windows installation hacks

<!-- Please describe your changes on the following line: -->
This is based on the awesome work by @UK992 in #12573.

Basically, it downgrades GCC on Windows to 5.4 within the mingw environment. This is a stopgap until #11756 is completed and we no longer need mingw.

Fixes #12512

r? @metajack

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [ ] `./mach build -d` does not report any errors
- [ ] `./mach test-tidy` does not report any errors
- [ ] These changes fix #__ (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [ ] These changes do not require tests because _____

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

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

☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev

@bors-servo bors-servo merged commit 94c3431 into servo:master Jul 25, 2016
@highfive highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jul 25, 2016
@larsbergstrom larsbergstrom deleted the windows_deps branch September 29, 2016 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows builds with updated msys/mingw gcc segfault while building skia

4 participants