aboutsummaryrefslogtreecommitdiffstats
path: root/python
Commit message (Collapse)AuthorAgeFilesLines
* Package macos/Linux builds under release/debug dirsWilliam Lee2016-12-091-5/+5
| | | | | | | | | This commit updates the 'mach package' command to generate output under the target/release and target/debug directories when run on macOS and Linux. etc/ci/upload_nightly.sh has also been updated to upload packages from the release/debug directories.
* Cleanup path manipulations under macOS/LinuxWilliam Lee2016-12-091-31/+39
| | | | | Path manipulation has been changed from using '/' to using os.path functions for packaging code for macOS and Linux targets.
* Fix `./mach clippy`.Corey Farwell2016-11-081-1/+1
| | | | Fixes https://github.com/servo/servo/issues/13117.
* Auto merge of #14092 - kivikakk:sort-check, r=Wafflespeanutbors-servo2016-11-073-3/+31
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sort check on JSON <!-- Please describe your changes on the following line: --> Check that JSON keys are ordered, and that there's no duplicates (for `resources/prefs.json` and `resources/package-prefs.json`). --- <!-- 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] 🆕 `./mach test-tidy --self-test` does not report any errors - [x] These changes fix #12283 <!-- Either: --> - [x] There are tests for these changes 🎉 <!-- 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/14092) <!-- Reviewable:end -->
| * Check for JSON key non-duplication and orderYuki Izumi2016-11-073-3/+31
| |
* | Auto merge of #13930 - shinglyu:perf-llvmpipe, r=aneeshusabors-servo2016-11-071-25/+30
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use headless rendering for performance test <!-- 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. --> <!-- 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/13930) <!-- Reviewable:end -->
| * Use headless rendering for performance testShing Lyu2016-11-071-25/+30
| |
* | Auto merge of #14025 - iamrohit7:debug-assertions, r=Wafflespeanutbors-servo2016-11-051-10/+14
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add --with-debug-assertions flag for Mach 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. --> <!-- 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/14025) <!-- Reviewable:end -->
| * | Add --with-debug-assertions flag for MachRohit Burra2016-11-051-10/+14
| | |
* | | moving datetimestamping responsiblities from mach packageJake Goldsborough2016-11-031-9/+3
|/ / | | | | | | to CI upload, swaps semicolons for dashes
* | Auto merge of #13986 - iamrohit7:tidy-feature-attrs, r=Wafflespeanutbors-servo2016-11-013-0/+47
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make test-tidy check for alphabetical ordering of #![feature(...)] statements <!-- 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 #13954 (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. --> <!-- 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/13986) <!-- Reviewable:end -->
| * | Tests alphabetical ordering of #![feature(...)]Rohit Burra2016-11-012-0/+22
| | |
| * | Alphabetical order for #![feature(...)] in lib.rsRohit Burra2016-11-011-0/+25
| | |
* | | Auto merge of #13988 - dsprenkels:tidy-fncalls, r=Wafflespeanutbors-servo2016-10-313-1/+11
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test-tidy: Check for space between function name and `(` <!-- Please describe your changes on the following line: --> Disallow an extraneous space in a function call between the function name and the opening parenthesis in Rust code, while ignoring macro declarations. --- <!-- 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 --all` does not report any errors - [x] `./mach test-tidy --self` does not report any errors - [x] These changes fix #13980 <!-- Either: --> - [x] There are tests for these changes, these are written in - `python/tidy/servo_tidy_tests/rust_tidy.rs` - `python/tidy/servo_tidy_tests/test_tidy.py` - [ ] These changes do not require tests <!-- 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/13988) <!-- Reviewable:end -->
| * | | test-tidy: Check for space between function name and `(`Daan Sprenkels2016-10-313-1/+11
| |/ / | | | | | | | | | | | | | | | | | | | | | Disallow an extraneous space in a function call between the function name and the opening parenthesis in Rust code, while ignoring macro declarations. This commit fixes #13980.
* / / Reinstate the wpt lint checking.Ms2ger2016-10-311-6/+10
|/ /
* | Auto merge of #13945 - mmatyas:android_libcpp_change, r=larsbergstrom,aneeshusabors-servo2016-10-282-1/+33
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the Android build system A huge update for the Android build system, which makes the Android target build again. There are still some runtime issues, see #13154. CC & created by @larsbergstrom. Fixes #12562. <!-- 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/13945) <!-- Reviewable:end -->
| * | Update to use the new Android library build rules.Mátyás Mustoha2016-10-282-1/+33
| | |
* | | Auto merge of #13558 - ↵bors-servo2016-10-281-2/+2
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | aneeshusa:fix-path-to-browserhtml-in-linux-runservo.sh, r=shinglyu Fix path to browserhtml in Linux runservo.sh <!-- 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 #13552 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [x] These changes do not require tests because needs to be verified manually <!-- 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/13558) <!-- Reviewable:end -->
| * | Fix path to browserhtml in Linux runservo.shAneesh Agrawal2016-10-031-2/+2
| | |
* | | Don't check untracked file in tidyShing Lyu2016-10-281-4/+1
| |/ |/|
* | Fix style unit testsManish Goregaokar2016-10-181-1/+3
| |
* | Auto merge of #13614 - jdm:wptrunnerup3, r=Ms2gerbors-servo2016-10-181-6/+9
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Upgrade wptrunner This gets us back to a pristine local copy and allows us to start experimenting with webdriver tests. <!-- 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/13614) <!-- Reviewable:end -->
| * | Avoid #12321 by creating environment before switching directories.Josh Matthews2016-10-061-6/+7
| | |
| * | Rename mach command to upgrade wptrunner and make it work more than once.Josh Matthews2016-10-061-1/+3
| | |
* | | Auto merge of #13432 - shinglyu:layout_viewer_clean, r=mbrubeckbors-servo2016-10-171-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Restore the layout trace viewer <!-- Please describe your changes on the following line: --> This patch restores the layout trace viewer removed in https://github.com/servo/servo/commit/e7510ab90c3482233ca8b188d748216901041970 , 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. --> <!-- 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/13432) <!-- Reviewable:end -->
| * | | Restore the layout trace viewerShing Lyu2016-10-171-1/+1
| | | |
* | | | Implement ./mach test-unit --bench.Bobby Holley2016-10-161-3/+5
|/ / /
* | | Remove test-geckolib, run test-stylo on travisManish Goregaokar2016-10-121-11/+0
| | |
* | | Run bindings tests with whole crateManish Goregaokar2016-10-121-2/+10
| | |
* | | Warn and exit mach if it detects MinGW PythonUK9922016-10-101-0/+8
| | |
* | | Auto merge of #13589 - splav:SVGElement#12974, r=pcwalton,Ms2gerbors-servo2016-10-071-0/+1
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support SVG element <!-- Please describe your changes on the following line: --> minimal SVG element implementation --- <!-- 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 #12974 (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. --> <!-- 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/13589) <!-- Reviewable:end -->
| * | add SVG2 draft link to WEBIDL_STANDARDS test-tidy listAlexandrov Sergey2016-10-071-0/+1
| | |
* | | Ensure that SERVO_ENABLE_DEBUG_ASSERTIONS is also used in build-cefLars Bergstrom2016-10-051-1/+10
|/ /
* | Auto merge of #13387 - emilio:debug-assertions, r=aneeshusa,jdm,pcwaltonbors-servo2016-10-041-0/+6
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Honor SERVO_ENABLE_DEBUG_ASSERTIONS on the build machines. <!-- Please describe your changes on the following line: --> As part of #13127. cc @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 <!-- 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/13387) <!-- Reviewable:end -->
| * | build: Honor SERVO_ENABLE_DEBUG_ASSERTIONS.Emilio Cobos Álvarez2016-10-021-0/+6
| | |
* | | Add mako to required python packagesJosh Matthews2016-10-031-0/+1
| |/ |/| | | It's required for the OSMesa build process.
* | Auto merge of #13447 - gterzian:check_webidls_folder_files, r=Wafflespeanutbors-servo2016-10-028-3/+47
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Have tidy ensure that the there are no extra files in the webidls folder #13427 <!-- Please describe your changes on the following line: --> Have tidy ensure that the there are no extra files in the webidls folder #13427 --- <!-- 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 #13427 (github issue number if applicable). <!-- Either: --> - [x] There are tests for these changes <!-- 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/13447) <!-- Reviewable:end -->
| * update docs with test commandGregory2016-10-021-0/+5
| |
| * tidy: checking directories for unexpected file extensionsGregory2016-10-027-3/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | update config, check_dir func to use config pass only_changed_files and exclude_dirs as args simplify config better conditional remove unused default, amend error message undo doc change
* | Auto merge of #13538 - mschmo:master, r=Wafflespeanutbors-servo2016-10-012-3/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid unecessary loop evaluation on empty lines for tidy check_license Refactored file line loop in `check_license()` to skip when a blank line is encountered. Only difference should be a very very small performance boost. Also changed the name of a test from the mispelling `test_licence()` to `test_license()` --- - [ ] `./mach build -d` does not report any errors (I'm not able to download the rust compiler on my laptop due to me probably just needing to update openssl: `Error downloading Rust compiler: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590). The failing URL was: https://static-rust-lang-org.s3.amazonaws.com/dist/2016-09-21/rustc-nightly-x86_64-apple-darwin.tar.gz`) - [X] `./mach test-tidy` does not report any errors <!-- Either: --> - [X] There are no added tests for this since `test_license()`, `test_shebang_license()` and `test_apache2_incomplete()` should suffice in this case. Though I'd be more than happy to add a test if requested. <!-- 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/13538) <!-- Reviewable:end -->
| * | using word license in test name for consistencyMatt Schmoyer2016-10-011-1/+1
| | |
| * | Avoid unecessary loop evaluation on empty lines for tidy check_licenseMatt Schmoyer2016-10-011-2/+3
| | |
* | | Ensure cargo is up to date before running update crate commandGuillaume Gomez2016-10-011-0/+2
|/ /
* | Auto merge of #13513 - nivekuil:master, r=Wafflespeanutbors-servo2016-09-301-13/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Return the error codes instead of implicitly returning None See https://github.com/servo/servo/issues/13344 --- <!-- 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 #13344 (github issue number if applicable). <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because refactoring without logic changes, 0 is falsey as None <!-- 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/13513) <!-- Reviewable:end -->
| * | Return the error codes instead of implicitly returning NoneKevin Liu2016-09-291-13/+4
| | |
* | | Auto merge of #12850 - Coder206:newFolderLinux, r=aneeshusabors-servo2016-09-301-18/+18
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | New folder linux <!-- 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 #11983 (github issue number if applicable). - [x] There are tests for these changes (./mach package) <!-- 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/12850) <!-- Reviewable:end -->
| * | | Use a temp folder when packaging on LinuxPatrick Trottier2016-09-301-18/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The purpose of the code is to make a new temporary folder with the necessary resources used to package Linux. Before this, the package was built based on a modified target directory. By using a temporary folder, it removes the need to rebuild Servo every time it gets packaged for Linux.
* | | | Auto merge of #13488 - vvuk:master, r=larsbergstrombors-servo2016-09-301-3/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use default console subsystem on Windows for dev builds <!-- Please describe your changes on the following line: --> If we keep the windows subsystem, we don't get any kind of log output. We should keep the normal console subsystem for dev 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/13488) <!-- Reviewable:end -->
| * | | | Use default console subsystem on Windows for dev buildsVladimir Vukicevic2016-09-281-3/+5
| | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-23 19:03:54 +0000