diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2016-07-18 22:34:18 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-07-18 22:34:18 -0700 |
commit | 2ce85f1cddb05689c231b1a17dfd2d374ad48377 (patch) | |
tree | 486ae5be60778bf5bf23ba652f97cd9a09c6a2d9 /components/script/dom/htmltrackelement.rs | |
parent | f88ecda3628ec2af3a4e4c7fdebeda47b4008d6f (diff) | |
parent | c30f7a3f69ed1ce40dff60e5329808f0cec16034 (diff) | |
download | servo-2ce85f1cddb05689c231b1a17dfd2d374ad48377.tar.gz servo-2ce85f1cddb05689c231b1a17dfd2d374ad48377.zip |
Auto merge of #12498 - upsuper:msvc-bindgen, r=emilio
Support generating bindings for MSVC
<!-- Please describe your changes on the following line: -->
This pull request includes several patches to the binding generating script to make it work with MSVC. The generated files are not updated because they may not be compatible with other platforms which the majority of developers are working on.
Only `regen.py` is modified. The two `.sh` files are not. Those files are very platform-specific and I don't think it's worth to make them work on Windows at all, and my hope is that we can get rid of those files and only use `regen.py`. I imagine the only left steps to get there are:
1. make clang version detectable via `bindgen` so that we get reliable clang version without needing to duplicate the library searching work (which has already been done in `clang-sys`)
2. checkout the git repo inside the python script and run `cargo build` there
BTW, it seems to me nightly Rust is not required to build `bindgen`, and thus we can probably get rid of the `multirust` detection. (Even if we need that, I think we should prefer `rustup` and optionally fallback to `multirust`. I know `rustup` is not yet available on Homebrew, though...)
---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [ ] These changes fix #__ (github issue number if applicable).
<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because it is a change to binding generating script
<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
r? @emilio
<!-- 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/12498)
<!-- Reviewable:end -->
Diffstat (limited to 'components/script/dom/htmltrackelement.rs')
0 files changed, 0 insertions, 0 deletions