aboutsummaryrefslogtreecommitdiffstats
path: root/python
Commit message (Collapse)AuthorAgeFilesLines
* Rewrite the ban-type lint in PythonJefry Lagrange2017-02-254-0/+54
| | | | | | Delete old rust ban lint and move tests to python tidy Fix ban lint regex and fix test
* Add 'use statements with extraneous spaces' tidy checkvwvww2017-02-233-0/+7
| | | | | | | | | | | | | | | | | | | | | Add 'use statements with extraneous spaces' tidy check I added simple check routine for 'use statements with extraneous spaces' and codes that breaks the check routine in rust_tidy.rs. * Added a code that using 'use statements with extraneous spaces' code in rust_tidy.rs * Added assertion code in test_tidy.py. * check_rust function in tidy.py now recognizes the simple case in the 'use statements with extraneous spaces'. * Ran tidy check on rust code and modified a code(tests/unit/style/parsing/inherited_text.rs) that is not passing on this new tidy check. TODO: this code has to be refactored to support more general cases. - [X] ./mach test-tidy does not report any errors - [X] These changes fix #14898 (github issue number if applicable).
* Auto merge of #15665 - UnICorN21:ds_store, r=SimonSapinbors-servo2017-02-221-2/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixed an issue related with .DS_Store <!-- 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 #15581. <!-- 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/15665) <!-- Reviewable:end -->
| * fixed an issue related with .DS_StoreHuxley2017-02-221-2/+2
| |
* | Kill the plugins crate and its clippy supportAnthony Ramine2017-02-211-21/+0
| | | | | | | | | | | | Sometimes clippy gets outdated by months, and its current support setup means that each Servo component need to opt into it by depending on the plugins crate manually, and not all components do that.
* | Disable LLVM assertions by default, on supported platforms.Simon Sapin2017-02-162-3/+12
|/ | | | But keep them on linux-dev CI.
* Allow disabling LLVM assertions in rustc (fixes #15548)Anthony Ramine2017-02-152-4/+12
|
* Switch to commit-specific nightliesAnthony Ramine2017-02-152-3/+4
|
* tidy: Ignore [replace]’d packages in the duplicate package lint.Simon Sapin2017-02-141-0/+2
|
* Tidy checks for styloManish Goregaokar2017-02-083-17/+32
|
* Update web-platform-tests and CSS tests.Ms2ger2017-02-061-1/+1
| | | | | - Update CSS tests to revision e05bfd5e30ed662c2f8a353577003f8eed230180. - Update web-platform-tests to revision a052787dd5c069a340031011196b73affbd68cd9.
* Remove freetype dependency on WindowsUK9922017-02-063-9/+6
|
* Option configuration to enable incremental compilation.Simon Sapin2017-02-051-0/+3
| | | | https://internals.rust-lang.org/t/incremental-compilation-beta/4721
* Find WPT files in Firefox repositoryGregory Szorc2017-02-031-7/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | The Servo repository is now (mostly) vendored in the Firefox Mercurial repository. For size and duplication reasons, the tests/wpt directory is not included in the vendored copy. This causes problems when running `mach` from the Firefox repository because `mach` references pip requirements files and module search paths from WPT. This commit adds code to detect when Servo's mach is running from a Firefox source tree and to resolve WPT paths to the Firefox location if appropriate. This enables `mach` to "just work" when running from the servo/ directory in the Firefox repository. The file looked for to identify the Firefox repository is identical to what Firefox's `mach` script uses. A potential issue with using Firefox's WPT files is that they may be different from those in the Servo repository and this could lead to differences in behavior - possibly even an error when loading/running `mach`. However, the behavior before this commit was that Servo's `mach` never worked in the Firefox repository (due to missing WPT files). And post-commit it does. So this seems like a "perfect is the enemy of good" scenario.
* Auto merge of #15138 - mdevlamynck:software-flag, r=jdmbors-servo2017-01-261-2/+11
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow running servo in software rendering mode from ./mach run <!-- Please describe your changes on the following line: --> This adds a flag to ./mach run to use software rendering on Linux (through the LIBGL_ALWAYS_SOFTWARE env var). --- <!-- 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 #14982. <!-- Either: --> - [X] These changes do not require tests because like said in the issue "There's no way to add automated tests for this". <!-- 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/15138) <!-- Reviewable:end -->
| * Allow running servo in software rendering mode from ./mach runMatthias Devlamynck2017-01-211-2/+11
| |
* | Make test-wpt-failure smoketest work again.Josh Matthews2017-01-241-9/+6
|/
* Auto merge of #14974 - aneeshusa:add-mach-bootstrap, r=metajackbors-servo2017-01-1915-442/+430
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use Salt for mach bootstrap <!-- Please describe your changes on the following line: --> This is currently WIP, but allows Salt for `mach bootstrap`. Not looking for review yet, just posting for visibility. You can run `./mach bootstrap` and Salt will run, letting you know what changes it would make for bootstrapping (doesn't actually run yet though). Currently, this reads from saltfs using gitfs, meaning it always tracks the master branch. (Note that this is blocked on https://github.com/servo/saltfs/pull/577 landing; in the meantime, I've included a small workaround in this PR to pull from my updated saltfs branch, which will need to be removed.) In the future, the relevant Salt code may be merged in tree as part of Docker support for TC, and the bootstrapper should be updated to read from in tree. Also, our Windows machines have not been Salted, so the existing Windows bootstrappers are retained. TODO: - [x] Hook into existing bootstrapping code less hackily - [x] Actually bootstrap instead of always using `test=True` mode (includes sudo) - [x] Default to interactive mode (test first, then ask), with a force flag - [x] Don't require running from the repository root directory - [x] Make it easy to add support for other distros --- <!-- 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 should 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/14974) <!-- Reviewable:end -->
| * Add a Salt bootstrapperAneesh Agrawal2017-01-174-4/+136
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Salt bootstrapper invokes Salt during `./mach bootstrap` to install Servo's build dependencies. It uses salt-call pinned to the same version of Salt as used in saltfs. Currently, the implementation uses gitfs and reads directly from the master branch of the saltfs repo; in the future this should be changed when the relevant Salt states are moved in-tree as part of Dockerization for TaskCluster. We have not Salted our Windows machines, so the existing Windows bootstrappers are retained. Currently this is only tested on Ubuntu Trusty. Salt uses various system python libraries, including `python-apt` on Debian-based OSes to interact with apt. `python-apt` does not seem to be installable via a requirements.txt file, and the versions available on PyPI are far behind the versions installed on actual Ubuntu machines. Additionally, adding `python-apt` as an unconditional python dependency would add bloat for users of other OSes, and lead to more churn as additional OSes are supported. However, as `python-apt` is already installed via apt on these machines, we can allow Salt to instead use the module by using `--system-site-packages` for the python virtualenv. We also add the `-I` flag to `pip install` to ensure we have a local, untouched copy of any other python packages used. However, because this prints system-level Python packages in scope, it slightly breaks isolation, so it is important to always pin all dependencies in the requirements files.
| * Clean up and simplify existing `mach bootstrap`Aneesh Agrawal2017-01-159-281/+137
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Default to interactive mode and remove the `--interactive` flag - Use `--force` to skip interactivity - Change MSVC dependency storage organization on disk: put each version into its own folder and directly refer to the versioned folders, providing immutability and making the installation list redundant - Reuse `host_triple()` function to fix broken bootstrapper dispatching - Simplify code: - Remove or inline many unused and redudant functions and variables - Prefer plain functions to classes - Consolidate into fewer files, remove unnecessary bootstrapper/ dir - Improve Python style - Sort dependency list
| * Avoid hardcoded references to rustc in download()Aneesh Agrawal2017-01-151-5/+5
| | | | | | | | | | | | | | Use the `desc` parameter instead to make the error messages customized for the actual download. Also use new-style format strings.
| * Create a util.py Python module for common functionsAneesh Agrawal2017-01-156-141/+160
| | | | | | | | | | Extracting these functions helps avoid circular dependencies, and make them easier to find/reuse.
| * Remove unusable --android flag for `mach bootstrap`Aneesh Agrawal2017-01-145-23/+4
| | | | | | | | `mach` can't do any bootstrapping for Android, so the flag is useless.
* | Tidy: Report invalid listed ignored directories and filesUK9922017-01-183-7/+42
| |
* | Moved tidy's BLOCKED_PACKAGES to the config fileMatthias Devlamynck2017-01-142-23/+8
|/
* Add config to do desktop notifications by running an abitrary command.Simon Sapin2017-01-131-14/+29
|
* Expand ~ in --with-gecko for ./mach build-geckolibManish Goregaokar2017-01-091-1/+1
|
* mach test-unit: also pass extra arguments to style_tests harnessSimon Sapin2017-01-091-0/+5
|
* Rust tidy: disallow assignment = in the beginning of lineJure Podgoršek2017-01-063-0/+6
|
* Package: Restore `run-servo` script on MacOSUK9922017-01-061-0/+10
|
* Package: Create ZIP on WindowsUK9922017-01-061-5/+43
|
* Added test-tidy check for blocked packages.Alan Jeffrey2017-01-051-0/+35
|
* Auto merge of #14706 - faineance:test_unit_no_capture, r=Wafflespeanutbors-servo2017-01-031-3/+9
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow passing --nocapture argument to test-unit mach command This adds the ` --nocapture` argument to the test-unit mach command. Defaulting to false, when given it passes `-- --nocapture` to cargo test to show stdout during test-unit runs. <!-- 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 #14595 <!-- 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/14706) <!-- Reviewable:end -->
| * Add nocapture argument to test-unitfaineance2017-01-031-3/+9
| |
* | Fix browserhtml pathRavi Shankar2017-01-031-2/+2
| |
* | Move get_browserhtml_path to command_base #14815Teodor Szente2017-01-023-14/+12
| | | | | | | | | | | | Remove unused import of find_dep_path_newest #14815 Move get_browserhtml_path top along with find_dep_path_newest #14815
* | Package: Small cleanupUK9922016-12-291-12/+9
| |
* | Package: Move Android packaging to target dirUK9922016-12-291-6/+12
| | | | | | | | Fixes #13800
* | Package: Set `shell.homepage` to browserhtmlUK9922016-12-291-18/+0
| | | | | | | | Closes #12154
* | Package: Simplify getting browserhtml pathUK9922016-12-291-16/+13
| |
* | Fix tidy json alphabetical order checkUK9922016-12-291-1/+1
| |
* | Package: Merge changed prefs from package-prefs.json to prefs.jsonUK9922016-12-291-15/+31
| |
* | Auto merge of #14346 - mati865:master, r=UK992,larsbergbors-servo2016-12-281-2/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use MSVC toolchain if PLATFORM is defined <!-- Please describe your changes on the following line: --> `VSInstallDir` exists only when Visual Studio is installed. `VS140COMNTOOLS` is defined by installing either Visual Studio or [Visual C++ Build Tools](http://landinghub.visualstudio.com/visual-cpp-build-tools) It will allow to build MSVC based Servo without having to install whole Visual Studio (few GiB). EDIT: Another (maybe cleaner) solution would be reverting https://github.com/servo/servo/commit/63ec96a57d4772a961bea63111c927ccd7c85a91 --- <!-- 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 `VS140COMNTOOLS` variable is already used in [mach.bat](https://github.com/servo/servo/blob/master/mach.bat) <!-- 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/14346) <!-- Reviewable:end -->
| * | use MSVC toolchain if PLATFORM is definedmati8652016-12-071-2/+4
| | |
* | | Auto merge of #14715 - UK992:tidy-check-lock, r=SimonSapinbors-servo2016-12-264-65/+72
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tidy: Check Cargo.lock for packages with same version and different sources <!-- Please describe your changes on the following line: --> r? @Wafflespeanut cc @SimonSapin --- <!-- 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 #14695 <!-- 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/14715) <!-- Reviewable:end -->
| * | | Tidy: Check Cargo.lock for packages with same version and different sourcesUK9922016-12-264-50/+61
| | | |
| * | | Tidy: Simplify path normalizationUK9922016-12-261-15/+11
| | |/ | |/|
* / | stylo: Copy binding files even if the build failed.Emilio Cobos Álvarez2016-12-251-1/+1
|/ / | | | | | | It's easier to stash the changes away than to find the generated files.
* | Auto merge of #14589 - servo:harden-script, r=jdm+Ms2gerbors-servo2016-12-163-0/+27
|\ \ | | | | | | | | | | | | | | | | | | | | | Reduce calls into arbitrary code with the ScriptThread::documents borrow held. <!-- 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/14589) <!-- Reviewable:end -->
| * | Add a tidy check for problematic match cases in script_thread.rsJosh Matthews2016-12-163-0/+27
| | |
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-30 12:41:25 +0000