Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Appveyor: Use ccache on Windows GNU builds #14850

Merged
merged 1 commit into from Jan 6, 2017
Merged

Conversation

@UK992
Copy link
Contributor

UK992 commented Jan 5, 2017

r? @larsbergstrom

Locally tested:

First run: (Build time: 26:27)

cache directory                     C:/github/servo/.servo/.ccache
primary config                      C:/github/servo/.servo/.ccache/ccache.conf
secondary config      (readonly)    C:/building/msys64/mingw64/etc/ccache.conf
cache hit (direct)                     0
cache hit (preprocessed)               0
cache miss                           953
called for link                        5
called for preprocessing               9
compile failed                         6
preprocessor error                    30
unsupported source language            1
autoconf compile/link                 43
no input file                          6
files in cache                      2523
cache size                         443.0 MB
max cache size                       5.0 GB

Second run: (Build time: 15:30)

cache directory                     C:/github/servo/.servo/.ccache
primary config                      C:/github/servo/.servo/.ccache/ccache.conf
secondary config      (readonly)    C:/building/msys64/mingw64/etc/ccache.conf
cache hit (direct)                   922
cache hit (preprocessed)              31
cache miss                           953
called for link                       10
called for preprocessing              18
compile failed                        12
preprocessor error                    60
unsupported source language            2
autoconf compile/link                 86
no input file                         12
files in cache                      2523
cache size                         443.0 MB
max cache size                       5.0 GB


This change is Reviewable

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Jan 5, 2017

Let's give this a try and see how it works out for us! I was hesitent because in the past we've run into cache size limits on AppVeyor, but it looks like they increased the Pro subscription cache size to 5GB:
https://www.appveyor.com/docs/build-cache/#cache-size-beta

@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Jan 5, 2017

📌 Commit 4f470f1 has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Jan 5, 2017

Testing commit 4f470f1 with merge f71ebc8...

bors-servo added a commit that referenced this pull request Jan 5, 2017
Appveyor: Use ccache on Windows GNU builds

r? @larsbergstrom

Locally tested:

First run: (Build time: 26:27)
```
cache directory                     C:/github/servo/.servo/.ccache
primary config                      C:/github/servo/.servo/.ccache/ccache.conf
secondary config      (readonly)    C:/building/msys64/mingw64/etc/ccache.conf
cache hit (direct)                     0
cache hit (preprocessed)               0
cache miss                           953
called for link                        5
called for preprocessing               9
compile failed                         6
preprocessor error                    30
unsupported source language            1
autoconf compile/link                 43
no input file                          6
files in cache                      2523
cache size                         443.0 MB
max cache size                       5.0 GB
```

Second run: (Build time: 15:30)

```
cache directory                     C:/github/servo/.servo/.ccache
primary config                      C:/github/servo/.servo/.ccache/ccache.conf
secondary config      (readonly)    C:/building/msys64/mingw64/etc/ccache.conf
cache hit (direct)                   922
cache hit (preprocessed)              31
cache miss                           953
called for link                       10
called for preprocessing              18
compile failed                        12
preprocessor error                    60
unsupported source language            2
autoconf compile/link                 86
no input file                         12
files in cache                      2523
cache size                         443.0 MB
max cache size                       5.0 GB

```

<!-- 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/14850)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jan 5, 2017

💔 Test failed - linux-rel-wpt

@jdm
Copy link
Member

jdm commented Jan 6, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Jan 6, 2017

Testing commit 4f470f1 with merge b97a753...

@bors-servo
Copy link
Contributor

bors-servo commented Jan 6, 2017

💔 Test failed - linux-rel-css

@jdm
Copy link
Member

jdm commented Jan 6, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Jan 6, 2017

Testing commit 4f470f1 with merge 6d4ccab...

bors-servo added a commit that referenced this pull request Jan 6, 2017
Appveyor: Use ccache on Windows GNU builds

r? @larsbergstrom

Locally tested:

First run: (Build time: 26:27)
```
cache directory                     C:/github/servo/.servo/.ccache
primary config                      C:/github/servo/.servo/.ccache/ccache.conf
secondary config      (readonly)    C:/building/msys64/mingw64/etc/ccache.conf
cache hit (direct)                     0
cache hit (preprocessed)               0
cache miss                           953
called for link                        5
called for preprocessing               9
compile failed                         6
preprocessor error                    30
unsupported source language            1
autoconf compile/link                 43
no input file                          6
files in cache                      2523
cache size                         443.0 MB
max cache size                       5.0 GB
```

Second run: (Build time: 15:30)

```
cache directory                     C:/github/servo/.servo/.ccache
primary config                      C:/github/servo/.servo/.ccache/ccache.conf
secondary config      (readonly)    C:/building/msys64/mingw64/etc/ccache.conf
cache hit (direct)                   922
cache hit (preprocessed)              31
cache miss                           953
called for link                       10
called for preprocessing              18
compile failed                        12
preprocessor error                    60
unsupported source language            2
autoconf compile/link                 86
no input file                         12
files in cache                      2523
cache size                         443.0 MB
max cache size                       5.0 GB

```

<!-- 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/14850)
<!-- Reviewable:end -->
@bors-servo bors-servo merged commit 4f470f1 into servo:master Jan 6, 2017
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
@UK992 UK992 deleted the UK992:appveyor-ccache branch Jan 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.