-
-
Notifications
You must be signed in to change notification settings - Fork 205
chore: adding support for x86_64-darwin #1310
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
Conversation
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.
Do we need to build the artifacts as part of the CI builds? Having support for being able to do it sounds good though
It does help us to run these builds/checks. Usually a true full build only happens when something has changed in the source, otherwise it is passed over. Then, when some source is changed, the build for that part will happen, the results will be cached into our binary cache, and so then 90% of the rest of the future ci runs will be able to deterministically refer to and use the cached version. In the case of this build, adding a new system did trigger a full build, but that is also what we needed to test, and achieve. So it is typical practice in nix based systems to run both builds and checks like this in ci. It should actually trigger a full build only in the small minority of cases. And in those cases, you almost always want to test the build, and cache it if it succeeds. |
* chore: adding support for x86_64-darwin * fix: handle logic for various darwin platforms * fix: != &&, == || * fix: support system here too * feature: support added for x86_64-darwin * fix: increase timeout so that aws access does not expire * fix: duration was too high * chore: see if 2 hours will work * fix: try more than 2 less than 3 hrs * chore: try a bit lower * chore: bump down lower * chore: set to the apparent max * chore: limit nix build to 15 on x86_64-darwin --------- Co-authored-by: Sam Rose <[email protected]>
What kind of change does this PR introduce?
Minimum basis for supporting x86_64-darwin build