-
Notifications
You must be signed in to change notification settings - Fork 38.7k
appveyor: script improvement #14241
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: script improvement #14241
Conversation
Reviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
|
|
utACK af1dcecefa09c43ca8fb50203d1b3f439cf33291
|
.appveyor.yml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be better to replace the install+upgrade with
>vcpkg remove --recurse --outdated
>vcpkg install %ALL_PACKAGES%
Otherwise, you'll be rebuilding packages that you no longer use (like boost interprocess).
You should also run ./bootstrap-vcpkg.bat after doing a git pull, because there may be important tool changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome
My original thought is that we could get rid of boost-interprocess by clear the cache, but if we use the command you provide, we don't have to clear the cache anymore. Just have to wait for next boost-interprocess update and it would be removed automatically.
59a50c2 appveyor: trivial build cache modifications (Chun Kuan Lee) Pull request description: - Reduce cache size from 1.55GB to 170MB by disabling whole program optimization. - The `*.iobj` `*.ipdb` cache is no longer required because of above change. - Clearing cache hit stats instead of deleting clcache stats.txt file - Enable access time on Windows so clcache would work correctly - Reduce max cache size from 2GB to 512MB (Uncompressed) - Remove unnecessary `boost-interprocess` vcpkg package - Split some command into multiple lines to make it easier to know what commands do. Tree-SHA512: 070b3ee05cd4323cef6fdab5ca0f223a5aa178f09b64d41fa39d0495f9bf3a387eadd10f5b72a2cd5b6f91579f19f595450fcf0afc05d7c5f6059a79fefaadc0
*.iobj*.ipdbcache is no longer required because of above change.boost-interprocessvcpkg package