Граф коммитов

106 Коммитов

Автор SHA1 Сообщение Дата
Aneesh Agrawal 84b6763a98 servo: Merge #15284 - Remove unused builders (from aneeshusa:remove-unused-windows-builders); r=larsbergstrom
These have been superseded by the windows-gnu and windows-msvc versions.
Removing them allows freeing up disk space on the Windows builders.

r? @larsbergstrom

<!-- Please describe your changes on the following line: -->

---
<!-- 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
- [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
- [ ] 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. -->

Source-Repo: https://github.com/servo/servo
Source-Revision: 9cdf78d53d0a101d99e911088a00c677476f5bf1
2017-01-30 07:25:28 -08:00
UK992 e730825162 servo: Merge #15243 - Upload MSVC nightlies via MSYS environment (from UK992:windows-builder); r=larsbergstrom
Follow-up to https://github.com/servo/saltfs/pull/572

r? @larsbergstrom

Source-Repo: https://github.com/servo/servo
Source-Revision: bd72da5f02f2e2fc7ff3bbdc9cb54917a635c388
2017-01-27 05:15:14 -08:00
Aneesh Agrawal 1f43cccad3 servo: Merge #15029 - Check all constellation files for panics (from aneeshusa:check-whole-constellation-for-panics); r=asajeffrey
Teaches the `etc/ci/check_no_panic.sh` script to handle directories,
so it can check all constellation files for panics.

<!-- Please describe your changes on the following line: -->

r? @asajeffrey
There are currently 4 `unwrap()`s in `components/constellation/timer_scheduler.rs`; I'm not sure how you want to handle those.

---
<!-- 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
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix #14976 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [x] These changes do not require tests because it adds more testing coverage

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

Source-Repo: https://github.com/servo/servo
Source-Revision: f9505c7fafcbabd24cfa613493ecacd3e29d273e
2017-01-20 12:10:32 -08:00
Aneesh Agrawal 4a4f05025d servo: Merge #15005 - Fix git author setting for brew upload (from aneeshusa:fix-git-author-setting-for-brew-upload); r=jdm
<!-- Please describe your changes on the following line: -->

The fix from #14935 wasn't quite right.

cc @paulrouget

---
<!-- 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
- [x] `./mach test-tidy` does not report any errors
- [x] These changes fix servo/saltfs#575 properly (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [x] These changes do not require tests because they should be tested by running the mac-nightly builder

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

Source-Repo: https://github.com/servo/servo
Source-Revision: dc93a72997aefaeb04cdc058b01fdd2eb14cef8f
2017-01-13 06:33:56 -08:00
Shing Lyu 83f364121f servo: Merge #14937 - Read firefox path from environment variable for performance test (from shinglyu:stylo-perf); r=Manishearth
<!-- Please describe your changes on the following line: -->
This is for Stylo performance testing (at least before we got Talos running)

r?@Manishearth

---
<!-- 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 independent from servo itself

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

Source-Repo: https://github.com/servo/servo
Source-Revision: da25e881459084395b3f8e5891d08de0645b5f0b
2017-01-11 23:14:48 -08:00
Paul Rouget b252a1118f servo: Merge #14935 - Add git author (from paulrouget:gitAuthor); r=aneeshusa
fix https://github.com/servo/saltfs/issues/575

Source-Repo: https://github.com/servo/servo
Source-Revision: 124301cf484818a87ab0b3e0c4ef34645be66174
2017-01-09 23:43:12 -08:00
Paul Rouget 55fe4d2c1f servo: Merge #14403 - Fix brew formula update (from paulrouget:fixBrew); r=aneeshusa
Fix https://github.com/servo/saltfs/issues/535

I decided to merge update_brew within upload_nightly.

I hope this will fix the brew formula upadte.

Source-Repo: https://github.com/servo/servo
Source-Revision: 207f9a5d3bb59cfc20b4455531325a3eb5a5ccf3
2017-01-05 19:59:18 -08:00
Shing Lyu 72f401280b servo: Merge #14124 - Flexbox trace (from shinglyu:flexbox-trace); r=glennw
<!-- Please describe your changes on the following line: -->
This is a follow up for #13740, so r? @jdm

The first patch enables JSON serialization for flexbox flows, the second one fixed format incompatibilities for the layout viewer.

The 3rd and 4th patches are just layout viewer UI enhancements, we could split that to a spearate PR if you prefer.

---
<!-- 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
- [x] These changes fix #13846  (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [x] These changes do not require tests because it's a trivial debug tool

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 58fec2f05a21eb69899c8ec21e4b95000cfed2f2
2016-12-28 00:33:31 -08:00
Aneesh Agrawal bb916ce698 servo: Merge #14670 - Rename Windows nightly builders for consistency (from aneeshusa:fix-windows-nightly-builder-names); r=larsbergstrom
<!-- Please describe your changes on the following line: -->

Fixup for #14651. r? @larsbergstrom

---
<!-- 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
- [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 these builders aren't used yet

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 5573f7cea3cc24e76501a05ff94b59c870ce7b82
2016-12-22 10:10:08 -08:00
Lars Bergstrom 72cbd6e04c servo: Merge #14651 - Create windows-gnu and windows-msvc builds (from larsbergstrom:add_msvc); r=aneeshusa
r? @metajack @edunham

The idea here is that I'm adding `-gnu` and `-msvc` variants in advance of the buildbot changes (in https://github.com/servo/saltfs/pull/491). Once those land and are pushed out to the buildbot master, it'll pick up these changed files and then we can remove the unsuffixed `windows-dev` and `windows-nightly`.

Source-Repo: https://github.com/servo/servo
Source-Revision: 25f32c4513a9d7c3514c4079eef35b1cc5c6b286
2016-12-22 08:47:15 -08:00
Lars Bergstrom 58f494133d servo: Merge #14587 - Exempt __cxa_type_match as an allowed dynamic symbol (from larsbergstrom:android_exempt_addition); r=Ms2ger
<!-- Please describe your changes on the following line: -->
This symbol is present in the new LLVM C++ stdlib, which is loaded dynamically from a .so file during startup.

Fixes #14571

r? @Ms2ger
---
<!-- 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. -->

Source-Repo: https://github.com/servo/servo
Source-Revision: 7c7dd0f965c53de4557aa8ad1762f2314b6036e8
2016-12-14 08:36:53 -08:00
William Lee 4269bf24d5 servo: Merge #13845 - Part of #13551 - Create packages under release/debug directories as appropriate (from birryree:package-under-target-profile-dirs); r=aneeshusa
r? @aneeshusa

This change implements fixes for #13551 for Linux and MacOS targets.

`/python/servo/package_commands.py` was modified so that:
- On MacOS, it creates all intermediate packaging directories like `dmg`, `brew`, and `brew-tmp` under `target/(release|debug)`, rather than in `target` directly.
- On MacOS, all packaging artifacts (`.dmg`, brew `.tar.gz`) are packaged under `target/(release|debug)`, rather than in `target` directly.
- On Linux, the resulting `tar.gz` Servo package is placed under `target/(release|debug)`, rather than in `target`.
- Also did some extra cleanup around path parsing in the MacOS packaging code, to use `os.path` methods rather than straight `'/'` parsing with `split` and `join` where it was applicable.

`/etc/ci/upload_nightly.sh` was modified to:
- Look for artifacts in `target/release` for `mac`, `macbrew`, and `linux` platforms, rather than just `target/`.

---

<!-- 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
- [x] These changes fix #13551
- [x] These changes do not require tests because it is based on CI and packaging tools. They were manually tested for correctness.

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

…lease/debug as specified or detected. Modify macos packaging to create all packages under release/debug directory. Updated etc/ci/upload_nightly.sh to support uploading from either release/debug directory, depending on what was built

Source-Repo: https://github.com/servo/servo
Source-Revision: 4cb3404c09d0b6a5a19a2c842d2e8809fae31e87
2016-12-09 14:46:14 -08:00
Corey Farwell fae587d847 servo: Merge #14499 - Add uncommitted lockfile change; fix check script. 🔒 (from frewsxcv:lockfile); r=jdm
Source-Repo: https://github.com/servo/servo
Source-Revision: 51a5a3402ced7a330c2a7d377dc4459792a28bee
2016-12-08 13:29:07 -08:00
Shing Lyu da07781c74 servo: Merge #14147 - Added an option to allow submitting test-perf result to perfherder (from shinglyu:test-perf-submit); r=larsbergstrom,aneeshusa
<!-- Please describe your changes on the following line: -->

This patch enables us  to run `./mach test-perf --submit` in CI to submit the result to perfherder. r? @aneeshusa

---
<!-- 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 too many manual setup required to test it

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 7d66bd710cee081155487ef36fe5b490e181c780
2016-11-30 19:11:16 -08:00
Shing Lyu b9f3e6e330 servo: Merge #14194 - Use Selenium for Gecko performance test (from shinglyu:gecko-webdriver); r=larsbergstrom,aneeshusa,jgraham
<!-- Please describe your changes on the following line: -->
We run the `etc/ci/performance` test on Gecko for comparison, but the old add-on approach doesn't work anymore. I'm using selenium to drive Gecko instead. Also did some refactoring and fixed some nits I saw along the way.

r? @aneeshusa

---
<!-- 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
- [x] These changes fix #14148  (github issue number if applicable).

<!-- Either: -->
- [x] 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. -->

Source-Repo: https://github.com/servo/servo
Source-Revision: 69f914b8b567424c85f9a947b453e7e9b0d4d243
2016-11-28 20:22:05 -08:00
Manish Goregaokar cedec048be servo: Merge #14400 - Add a log of only intermittents (from Manishearth:filter-intermittents); r=KiChjang
r? @KiChjang

Source-Repo: https://github.com/servo/servo
Source-Revision: 015c7642618e5be3df06a8799a87ec152f7d46fc
2016-11-28 18:38:53 -08:00
Manish Goregaokar 62888fa987 servo: Merge #14331 - Add ./mach filter-intermittents for catching intermittents on CI (from Manishearth:filter-intermittents); r=larsbergstrom,jdm
cc @metajack

The plan here is to run this on the error summary of the logs after each WPT/CSS run, as `./mach filter-intermittents wpt-errorsummary.log --output filtered-errorsummary.log --auth /path/to/authfile`

The `filtered-errorsummary.log` file will be a buildbot artifact for this run, and can be used for updating test results.

We change WPT/CSS runs to not cause test failures on Buildbot; instead; the test failure will be caused by this job.

We should at some point add a separate highfive task which ccs the appropriate bugs and tracks consistent failures. (We can change mach filter-intermittents to output a second file of intermittents matched to bug numbers to make this easy)

A small issue with this is that this simple task might mask failures of the WPT harness itself. We have a separate test that tests if the test harness works though. (This test will fail if the log files it requires don't exist, perhaps that's enough to solve the problem)

r? @larsbergstrom

Source-Repo: https://github.com/servo/servo
Source-Revision: 4021d057c941f38ca9f9ef9bb03f91d1b54f01c0
2016-11-28 15:18:04 -08:00
Shing Lyu f7e0070425 servo: Merge #13930 - Use headless rendering for performance test (from shinglyu:perf-llvmpipe); r=aneeshusa
<!-- Please describe your changes on the following line: -->

Properly set the software rendering environment variables and use `-z` to run the performance test in headless mode. Also changed some logging format to improve the readability and reduce log size.
---

<!-- 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
- [x] These changes fix #13903 (github issue number if applicable).

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

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

Source-Repo: https://github.com/servo/servo
Source-Revision: f5a3d68513456b991313dfac19b94d630cc326be
2016-11-07 01:24:06 -06:00
Rohit Burra 42d578b558 servo: Merge #14025 - Add --with-debug-assertions flag for Mach (from iamrohit7:debug-assertions); r=Wafflespeanut
I tested out building a release with the flag and added a `debug_assert!(false)` in `fn main` and it panicked.
<!-- 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
- [X] These changes fix #14009  (github issue number if applicable).

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

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

Source-Repo: https://github.com/servo/servo
Source-Revision: dbce416ad436b9a3a007181fbd5d0f158cd7d9e4
2016-11-05 22:57:44 -05:00
Jake Goldsborough 4fe7c0a9ad servo: Merge #14058 - moving datetimestamping responsibilities from mach package (from ducks:make-upload-nightly-put-datetimes-12128); r=Wafflespeanut
to CI upload, swaps semicolons for dashes

<!-- Please describe your changes on the following line: -->
This fixes #12128 by moving the datetimestamping responsibility from the python packaging script to the CI build shell script.

---
<!-- 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
- [X] These changes fix #12128 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because they are part of CI build

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 4c5d84d81a45a2f85ceca88ca69374b464722cd1
2016-11-04 04:04:34 -05:00
Shing Lyu 768804b9d4 servo: Merge #13829 - Layout viewer fixes (from shinglyu:layout_viewer); r=mbrubeck
<!-- Please describe your changes on the following line: -->

This is a followup for #13432. This patch should be landed after #13740 to accommodate the change in JSON format in #13740. It also included some UI layout fixes to make the UI more readable. I'm splitting the commits for easier review, I'll squash them after a r+.
---

<!-- 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's just a external debug tool

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 524ed827007dee1563aed25f3bc3d0edff77ea78
2016-11-02 18:34:26 -05:00
6br 0d518cda36 servo: Merge #10942 - Rename check_no_unwrap.sh to check_no_panic.sh (from 6br:rename_check_no_panic); r=aneeshusa
Hello, I fixed #10930.

Source-Repo: https://github.com/servo/servo
Source-Revision: 3ae0a3335dbaf3f67c775a1853941ed538d97563

--HG--
rename : servo/etc/ci/check_no_unwrap.sh => servo/etc/ci/check_no_panic.sh
2016-10-29 00:22:56 -05:00
Aneesh Agrawal 991d99e061 servo: Merge #13968 - Remove steps for unused linux-dev-yaml test builder (from aneeshusa:remove-steps-for-yaml-test-builders); r=larsbergstrom
<!-- Please describe your changes on the following line: -->

r? @larsbergstrom

---
<!-- 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
- [x] These changes do not require tests because that is #13838

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

These steps were used while testing the transition to dynamic steps.
Now that dynamic steps are working properly, this builder has been
decommissioned, and its steps are no longer needed.

Source-Repo: https://github.com/servo/servo
Source-Revision: e4fcc066d1248dc072026ba1dd887ffca5049c53
2016-10-28 18:59:16 -05:00
Mátyás Mustoha 5d3c509519 servo: Merge #13946 - Update `etc/ci/check_dynamic_symbols.py` (from mmatyas:android_dyncheck); r=larsbergstrom
Update `etc/ci/check_dynamic_symbols.py` to work after [the latest changes](https://github.com/servo/servo/pull/13945) in the Android build system.

Part of #13154.

Source-Repo: https://github.com/servo/servo
Source-Revision: b661f0691611c8c568840b75ac646af960aa683d
2016-10-28 14:33:13 -05:00
Shing Lyu 1a39b2a083 servo: Merge #13432 - Restore the layout trace viewer (from shinglyu:layout_viewer_clean); r=mbrubeck
<!-- Please describe your changes on the following line: -->
This patch restores the layout trace viewer removed in e7510ab90c , with some changes that makes it work with https://github.com/servo/servo/pull/13214

Sorry for the minified 3rd party CSS frameworks, it was from the original version and I haven't got time to replace them with hand-crafted CSS.

---
<!-- 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 is a stand-alone debugging tool

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 0af9be94298c676f5bf7881847880e6f43955602
2016-10-17 10:06:50 -05:00
Manish Goregaokar 016ff30ae6 servo: Merge #13720 - Add release test-stylo to buildbot steps, https://github.com/servo/saltfs/pull/508#issuecomment-253203920 (from Manishearth:resync-yml); r=aneeshusa
r? @aneeshusa

Source-Repo: https://github.com/servo/servo
Source-Revision: 4034bd68c2b2b9b899ecfb6e0d2b3ba17109c5ae
2016-10-12 17:03:48 -05:00
Aneesh Agrawal 4d0bfee5fc servo: Merge #13611 - Sync Buildbot steps config from saltfs (from aneeshusa:migrate-steps-from-saltfs); r=Ms2ger
<!-- 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
- [x] These changes are part of https://github.com/servo/saltfs/issues/316

<!-- Either: -->
- [ ] There are tests for these changes OR
- [x] These changes do not require tests because they just copy the steps configuration

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

Now that our Buildbot configuration is successfully able to read the
steps configuration from the main servo repo and run builds, sync the
steps in the servo tree to match the latest steps.yml from the saltfs
repo.

Source-Repo: https://github.com/servo/servo
Source-Revision: ff04558ce26203cc3eb63edc9a15a6e6622b814a
2016-10-06 15:15:00 -05:00
Alan Jeffrey 208cea2587 servo: Merge #13472 - Don't log in the middle of mutating a reentrant lock (from asajeffrey:util-remutex-dont-log-while-mutating-lock); r=jdm
<!-- Please describe your changes on the following line: -->

Moved assertion that we are the lock owner to after the lock release. The problem is that Servo uses a reentrant lock for logging, so logging in the middle of mutating the lock is A Bad Idea.

While I was at it, I tidied up the reentrant lock code.

cc @jdm

---
<!-- 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
- [X] These changes fix #13463.
- [X] These changes do not require tests because they fix an existing intermittent.

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 318b23ed0047cc39625ca2f33a55d647b5886019
2016-10-04 16:01:40 -05:00
Aneesh Agrawal fc6b0f7463 servo: Merge #13507 - Don't leak GitHub tokens when pushing (from aneeshusa:avoid-leaking-github-tokens); r=SimonSapin
<!-- Please describe your changes on the following line: -->

---
<!-- 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
- [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 they just remove output/were lightly tested by hand

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

If git is unable to resolve the repo address (which includes the token),
it will print a message to stderr with the path to the repo, thus
leaking the token. Avoid doing this.

Source-Repo: https://github.com/servo/servo
Source-Revision: d067f469d07ced138f9a63d1bdfc83fbc201a62f
2016-10-04 08:59:11 -05:00
Patrick Walton a2cdb30f64 servo: Merge #13519 - profile: Integrate the time profiler with the macOS signpost infrastructure (from pcwalton:signpost); r=larsbergstrom
With this change, if you supply the `-Z signpost` flag, Instruments.app
can display Servo events overlaid with callstack data. Even better, you
can get call stack profiling for individual Servo events (one layout,
one network request, one style recalculation, etc.)

This adds a dependency on the `signpost` crate. On unsupported OS's,
this crate is an no-op.

Screenshot: http://i.imgur.com/agLyzOU.png

r? @larsbergstrom

Source-Repo: https://github.com/servo/servo
Source-Revision: 7f84982de2fe8283d1df37d704842e037432087e
2016-10-01 04:18:56 -05:00
Josh Matthews fb0f735d96 servo: Merge #13454 - Fix destination of cp in doc build (from servo:jdm-patch-5); r=aneeshusa
Fixes #12964.

Source-Repo: https://github.com/servo/servo
Source-Revision: 89301826f4a879524852899ac592720a78dd1716
2016-09-29 02:18:51 -07:00
Manish Goregaokar cda519b264 servo: Merge #13448 - Fix doc build (fixes #12964) (from Manishearth:fx-doc); r=Ms2ger
r? @Ms2ger

Source-Repo: https://github.com/servo/servo
Source-Revision: 2474393d15b4e6259bdcf366d3423c3a6cd2eaac
2016-09-27 03:35:58 -05:00
Josh Matthews 34a1fa3299 servo: Merge #13410 - Fix doc build by resurrecting generation of supported DOM APIs.h (from jdm:doc_fix); r=Ms2ger
This should fix #12964.

Source-Repo: https://github.com/servo/servo
Source-Revision: 4ceea0426c933c853b3479acaa3e523c2b34a467
2016-09-26 23:33:53 -05:00
Alan Jeffrey 31881f012e servo: Merge #13248 - Added etc/ci/performance downloads to .gitignore (from asajeffrey:etc-ci-performance-gitignore); r=shinglyu
<!-- Please describe your changes on the following line: -->

Added the files downloaded for `etc/ci/performance` to `.gitignore`, which gets rid of lot of spurious reporting from `git status`.

r? @shinglyu

---
<!-- 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
- [X] These changes do not require tests because it's about git

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 58e6d7bd58d039c5ba23fab37a592c8e976dbb36
2016-09-26 06:03:55 -05:00
CHANG FU CHIAO e44e098231 servo: Merge #13091 - Create ./mach test-perf command (from zack1030:issue-12792); r=shinglyu
<!-- Please describe your changes on the following line: -->

---
<!-- 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
- [X] These changes fix #12792 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because it is low-risk

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

in **testing_commands.py**, I linked the new command to **test_perf.sh**
in **test_perf.sh**, it prepares tp5n.zip and virtual environment
then runs **test_all.sh**

Source-Repo: https://github.com/servo/servo
Source-Revision: 99435663f8e986d1851e4aec683dbbad7d53c0a7
2016-09-26 04:36:04 -05:00
Alan Jeffrey 3b4dca5ac6 servo: Merge #12946 - Etc ci performance run locally (from asajeffrey:etc-ci-performance-run-locally); r=shinglyu
<!-- Please describe your changes on the following line: -->

Changes to `etc/ci/performance` to get it to run locally on my machine at least!

---
<!-- 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
- [X] These changes do not require tests because this is performance-testing code

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 9e349544f0fed306861814ad4b71239645d034ce
2016-09-06 04:07:05 -05:00
CHANG FU CHIAO a9994c72a2 servo: Merge #13073 - FIX TreeherderClient deprecated param (from zack1030:issue-13047); r=shinglyu
<!-- Please describe your changes on the following line: -->

---
<!-- 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
- [X] These changes fix #13047 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because it is low risk

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

According to warning from TreeherderClient(),
the parameters `host` and `protocol` are deprecated.
Then use `server_url` instead.

Source-Repo: https://github.com/servo/servo
Source-Revision: 5aa95979f50ed044d6145150fa498211792f49da
2016-08-27 01:18:11 -05:00
Paul Rouget 2dfb348c21 servo: Merge #12688 - brew package (from paulrouget:brew); r=larsbergstrom
Addressed #12600

This is just a WIP PR to see if this approach is right.

The same way night builds are uploaded every day, we'd like to offer a brew binary package as well.

Step 1 is to package a tarball along the dmg. This is done in `package_commands.py`.
Step 2 is to upload the package. Done is `update_brew.sh`.
Step 3 is to update the brew Formula. [servo-bin.rb](https://github.com/paulrouget/homebrew-servo/blob/master/servo-bin.rb.in) is updated in `update_brew.sh`.
Step 4 is to trigger `update_brew.sh` along `update_nightly.sh`. https://github.com/servo/saltfs/blob/master/buildbot/master/files/config/steps.yml

Source-Repo: https://github.com/servo/servo
Source-Revision: a2ce4aa8525c3c83388c5401016a79ffe0444c03
2016-08-14 15:25:25 -05:00
Anthony Ramine 174065181a servo: Merge #12800 - Enable canvas_over_area.html (closes #8311) (from nox:intermittent); r=jdm
Source-Repo: https://github.com/servo/servo
Source-Revision: 7afe0376259e7460461410da03523ebec2f86c8f
2016-08-10 09:49:56 -05:00
Anthony Ramine 9facf66cad servo: Merge #12796 - Enable hide_layers2.html (closes #8769) (from nox:intermittent); r=jdm
Source-Repo: https://github.com/servo/servo
Source-Revision: dc47e66d8c955b2c84b780b3c93cf28e01fe6126
2016-08-10 08:05:12 -05:00
Michael Howell 5637a89704 servo: Merge #12629 - Intermittents (from notriddle:intermittents); r=aneeshusa
We probably want to have Buildbot run that script.

---

- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes fix #12435, #11574, and #11561
- [X] These changes do not require tests because reasons

Source-Repo: https://github.com/servo/servo
Source-Revision: 944cbbff8ba75b0fc493222923137be35c701aca

--HG--
rename : servo/python/tidy/servo_tidy_tests/test_ignored/whee/foo/bar.rs => servo/etc/ci/former_intermittents_wpt.txt
2016-08-09 23:28:17 -05:00
Shing Lyu fb11a939f3 servo: Merge #11107 - Bug 10452 - Page Load Time Test Runner (from shinglyu:servo-perf)
Source-Repo: https://github.com/servo/servo
Source-Revision: 7ed9134e5a8401380253f75d4a7ce43ab5027241
2016-08-08 16:22:50 +08:00
Aneesh Agrawal b707711b20 servo: Merge #12736 - Add more shell script lints (from aneeshusa:prohibit-backticks-in-shell-scripts); r=Wafflespeanut
<!-- Please describe your changes on the following line: -->

The "$(some_command arg1 arg2)" form is preferred to the
`some_command arg1 arg2` form because it nests unambiguously.
Add a lint for this to tidy.

---
<!-- 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: -->
- [x] 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. -->

Source-Repo: https://github.com/servo/servo
Source-Revision: aa900b91aa9417b6aa032b4eff878f2916c480f0
2016-08-06 21:57:31 -05:00
Jim Berlage 5b1825a0cf servo: Merge #12295 - Add linting for shell scripts (from jimberlage:12158/shell-linting); r=aneeshusa
<!-- Please describe your changes on the following line: -->

This changes tidy to check shell scripts for the proper shebang and
options.  It does not check that variables are formatted correctly.  It
also adds a check for the MPL 2.0 license in shell scripts.

---
<!-- 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
- [X] These changes fix #12158 (github issue number if applicable).

<!-- Either: -->
- [X] 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. -->

Source-Repo: https://github.com/servo/servo
Source-Revision: 4bc629b369242d16675161ab6af8141dab0bb556
2016-08-04 12:31:17 -05:00
Lars Bergstrom e02ed9e235 servo: Merge #12606 - Fix windows msi directory (from larsbergstrom:windows_msi_directory); r=metajack
<!-- Please describe your changes on the following line: -->
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. -->

Source-Repo: https://github.com/servo/servo
Source-Revision: f71b584458fcdd76465396854e204bb1e98d6553
2016-07-26 14:53:06 -05:00
Jack Moffitt 16fb113715 servo: Merge #11967 - Add Windows packaging to create an MSI installer (from metajack:windows-package); r=larsbergstrom
<!-- Please describe your changes on the following line: -->

---
<!-- 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 they require manual testing

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 2746c476d3114d987b153d93bed2d302ba2e52a5
2016-07-25 16:48:33 -05:00
Josh Matthews 3b513aab55 servo: Merge #12470 - Place apis.html in the same place as css-properties.html (from servo:jdm-patch-2); r=Manishearth
Right now it's visible at http://doc.servo.org/apis.html, unlike http://doc.servo.org/servo/css-properties.html.

Source-Repo: https://github.com/servo/servo
Source-Revision: d9b7c060a5ec7a6c0556de371b3e9ed3dc832337
2016-07-16 00:32:21 -07:00
Josh Matthews 75e683bbec servo: Merge #12451 - Generate a list of supported DOM APIs from parsed WebIDLs (from jdm:generate-dom-apis); r=Ms2ger
Supplement the existing list of supported CSS properties with an equivalent list of DOM APIs.

---
<!-- 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
- [X] These changes do not require tests because automatic generated documentation

Source-Repo: https://github.com/servo/servo
Source-Revision: 73ad0b928dd158be1eeba30dd21e67d4b9d82048
2016-07-15 15:13:22 -07:00
Aneesh Agrawal 5602842294 servo: Merge #12200 - Add linux dev yaml (from aneeshusa:add-linux-dev-yaml); r=larsbergstrom
<!-- Please describe your changes on the following line: -->

Sync `steps.yml` from saltfs and add `linux-dev-yaml` builder for testing steps.yml switchover. Helps with servo/saltfs#316.

---
<!-- 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
- [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 linux-dev-yaml is non-gated and other changes are running in production

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

Source-Repo: https://github.com/servo/servo
Source-Revision: a4b6705c02910d6b4dfcc0b2a9e2c909dfc89ef9
2016-07-03 19:24:28 -07:00
Jim Berlage 329b44edd2 servo: Merge #12156 - Adds explicit mime type to upload_nightly.sh (from jimberlage:12153/mime-type-s3); r=aneeshusa
<!-- Please describe your changes on the following line: -->
This changes the script to pass the `--mime-type` option to `s3cmd`, as
described [here](http://s3tools.org/usage).  The MIME type used is
`application/octet-stream`.

---
<!-- 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 #12153 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because this is a minimal change to the upload_nightly.sh script.

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

Source-Repo: https://github.com/servo/servo
Source-Revision: 1b8a2643d1047a2caf537d6180e464b6daf7b539
2016-07-02 09:57:23 -07:00