| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
'mach test' to be able to select the render-mode.
|
| |
|
|
|
|
|
|
|
| |
Fixes #6236
Also included in this commit are the changes need to make flake8 pass
for the existing python file
|
| |
|
|
|
|
|
| |
After #6268 merged, I found out that everything in the tree currently
passes the tidy checker.
|
|\
| |
| |
| |
| |
| | |
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6268)
<!-- Reviewable:end -->
|
| | |
|
| | |
|
|\ \
| |/
|/|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The existing code for setting up the environment assumes that
the directory layout containing rust and cargo conforms to the
one used by rust-installer's tarballs. This makes overriding
the system root awkward for simple cases where I want to test
my own build directly from the rust build directory. This
patch just adds a second path to PATH and LD_LIBRARY_PATH
to accomodate both disk layouts.
Conflicts:
python/servo/command_base.py
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6267)
<!-- Reviewable:end -->
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The existing code for setting up the environment assumes that
the directory layout containing rust and cargo conforms to the
one used by rust-installer's tarballs. This makes overriding
the system root awkward for simple cases where I want to test
my own build directly from the rust build directory. This
patch just adds a second path to PATH and LD_LIBRARY_PATH
to accomodate both disk layouts.
Conflicts:
python/servo/command_base.py
|
|/
|
|
|
|
| |
Trailing whitespace was found using flake8
This is working towards making flake8 part of the tidy process: #6236
|
|
|
|
|
|
|
| |
Versions of git before 1.8.1 do not support git submodule --recursive sync
This commit makes update_submodules() exit with an error message if the version is <1.8.1
https://github.com/servo/servo/issues/5637
|
|\
| |
| |
| |
| |
| | |
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6201)
<!-- Reviewable:end -->
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This might allow us to merge #5648
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6232)
<!-- Reviewable:end -->
|
| |/
| |
| |
| | |
This might allow us to merge #5648
|
|\ \
| | |
| | |
| | |
| | |
| | | |
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6234)
<!-- Reviewable:end -->
|
| |/ |
|
|/
|
|
| |
Found using flake8
|
|
|
|
| |
Part of https://github.com/servo/servo/issues/6041
|
| |
|
| |
|
|
|
|
| |
(And the same for `rr-record`.) For consistency with `mach build`.
|
| |
|
|\
| |
| |
| |
| |
| |
| |
| | |
Require either `--dev` or `--release`, unless a default build.mode is set in `.servobuild`. This is @jinankjain's patch from #5965, rebased onto master. It is already reviewed, and only waiting for servo/saltfs#28 to be deployed.
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6038)
<!-- Reviewable:end -->
|
| |
| |
| |
| |
| | |
Require either --dev or --release, unless a default build.mode is set
in .servobuild. Fixes #5933.
|
| | |
|
|/
|
|
|
| |
Also refactors the notification function to introduce some single
responsibility.
|
|
|
|
| |
Closes #5582
|
|\
| |
| |
| |
| |
| |
| |
| | |
This is part of #5965. It needs to land before the rest of that PR, so we can land servo/saltfs#28 without breaking automated builds for other PRs. r? @larsbergstrom
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6023)
<!-- Reviewable:end -->
|
| |
| |
| |
| |
| | |
This is part of #5965. It needs to land before the rest of that PR, so we can
land servo/saltfs#28 without breaking automated builds for other PRs.
|
| | |
|
|/
|
|
| |
Fixes #5929
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
The 300 second threshold was originally from the Gecko/Firefox build system.
It doesn't fit Servo builds, which are shorter, and often hover right around
the 300 second mark (making the notification unpredictable).
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Required by the latest version of the openssl-sys build script. Fixes #5145.
r? @larsbergstrom or @glennw
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/5784)
<!-- Reviewable:end -->
|
| |
| |
| |
| | |
Required by the latest version of the openssl-sys build script. Fixes #5145.
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Closes #5707. (Includes a rebase of it.)
Fixes #5688.
r? @mbrubeck
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/5776)
<!-- Reviewable:end -->
|
| | |
| | |
| | |
| | |
| | | |
Closes #5707. (Includes a rebase of it.)
Fixes #5688.
|
| | | |
|
| | | |
|
| | | |
|
| |/
|/| |
|
|/ |
|
| |
|
|
|
|
|
|
| |
platforms.
Incorporate suggestions from Code review. Change function name to lower case.
|
|
|
|
| |
The urllib version used a `FancyURLOpener` which use urllib. But urllib does not not handle proxies with SSL well.
A better solution would be to use `requests` but I prefer to stay with the "no-dependency" approach.
|
| |
|