Skip to content

mac-ci: add a CI workflow for Mac OS builds#1748

Merged
mattklein123 merged 6 commits intoenvoyproxy:masterfrom
turbinelabs:unified-circleci-build
Sep 27, 2017
Merged

mac-ci: add a CI workflow for Mac OS builds#1748
mattklein123 merged 6 commits intoenvoyproxy:masterfrom
turbinelabs:unified-circleci-build

Conversation

@zuercher
Copy link
Copy Markdown
Member

This configures Mac OS builds for envoy directly in the envoyproxy Circle CI account.

Per Circle CI this functionality should be enabled on that account.

Copy link
Copy Markdown
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Go with this or do you want to add back caching?

Copy link
Copy Markdown
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry just caught a typo. Otherwise LGTM.

ci/README.md Outdated
The MacOS CI build is part of the [CircleCI](https://circleci.com/gh/envoyproxy/envoy) workflow.
Dependencies are installed by the `ci/mac_ci_setup.sh` script, via [Homebrew](https://brew.sh),
which is pre-installed on the CircleCI MacOS image. The dependencies are cached are re-installed
n every build. The `ci/mac_ci_steps.sh` script executes the specific commands that
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo "n"

Signed-off-by: Stephan Zuercher <[email protected]>
@mattklein123 mattklein123 merged commit 0e75866 into envoyproxy:master Sep 27, 2017
costinm pushed a commit to costinm/envoy that referenced this pull request Oct 2, 2017
@zuercher zuercher deleted the unified-circleci-build branch October 25, 2017 22:37
jpsim pushed a commit that referenced this pull request Nov 28, 2022
jpsim pushed a commit that referenced this pull request Nov 29, 2022
mathetake added a commit that referenced this pull request Mar 3, 2026
**Description**

This will reduce the change surface required when (if ever) changing the
json lib dep

---------

Signed-off-by: Takeshi Yoneda <[email protected]>
Co-authored-by: Aaron Choo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants