-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Updated MacOSX workflow with caching #1238
Conversation
770c441
to
0088b67
Compare
b13a1e9
to
3995c82
Compare
f073a5e
to
3d1f79e
Compare
I don't think I'll be able to look closely at these PRs in the near future, sorry :( |
Ok, no problem. ;-) |
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.
this looks very verbose and complicated, but if this is what it takes to have fast caching with github-ci... well.
Unfortunately yes, we just have 5 GB of cache there and we want to cache the |
61ee2d1
to
3785d23
Compare
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.
Now should be fine, rebased it.
3785d23
to
1f09c46
Compare
@TheJJ rebased |
@TheJJ LGTM :-) |
on: [push, pull_request] | ||
env: | ||
QT_INSTALL_VERSION: '5.14.2' # This version we need to set, because we don't want 5.12.x LTS | ||
CCACHE_COMPRESS: 1 |
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.
Very wise
- name: Build | ||
run: make | ||
run: make build -j"$(sysctl -n hw.physicalcpu || echo 4)" |
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.
Better use ninja
(Outdated) |
Should be much faster now. And it builds with clang++ and ccache. 👍
ref.: #1196