Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upTurn on sccache logging #3278
Turn on sccache logging #3278
Conversation
| @@ -149,13 +149,21 @@ tasks: | |||
| git checkout {{event.head.sha}} | |||
| source $HOME/servotidy-venv/bin/activate | |||
| servo-tidy | |||
| sccache --stop-server || true | |||
This comment has been minimized.
This comment has been minimized.
luser
Nov 6, 2018
Contributor
You might want to stick a --stop-server invocation after your test steps as well, since sccache outputs stats after shutting down and those are useful to have in build logs.
|
This looks good to me. |
| @@ -149,13 +149,21 @@ tasks: | |||
| git checkout {{event.head.sha}} | |||
| source $HOME/servotidy-venv/bin/activate | |||
| servo-tidy | |||
| sccache --stop-server || true | |||
| export RUST_LOG=sccache=debug | |||
This comment has been minimized.
This comment has been minimized.
luser
Nov 6, 2018
Contributor
Actually, after looking at a build log on this PR, you probably don't want to export this, just put it in the --start-server line. Otherwise you get log spew every time cargo invokes sccache.
|
Here's sccache.log from the OS X debug tests task. |
|
Updated patch to address both comments |
|
@bors-servo r=luser (Having only one mac CI machine is really slowing things down, and apparently even that one is showing some errors, so getting this in sooner will help us debug what's going on) |
|
@staktrace: |
|
@bors-servo r+
…On Thu, Nov 8, 2018, 9:33 AM bors-servo ***@***.*** wrote:
@staktrace <https://github.com/staktrace>:
|
|
|
|
@bors-servo r- |
|
@bors-servo r=jrmuizel |
|
|
|
@bors-servo try- |
Turn on sccache logging This should help track down #3248, per discussion with @luser on IRC. r? anyone <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/3278) <!-- Reviewable:end -->
|
|
staktrace commentedNov 6, 2018
•
edited by larsbergstrom
This should help track down #3248, per discussion with @luser on IRC.
r? anyone
This change is