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

CI: MacOS arm build check #205

Merged
merged 20 commits into from
Feb 26, 2024
Merged

CI: MacOS arm build check #205

merged 20 commits into from
Feb 26, 2024

Conversation

678098
Copy link
Collaborator

@678098 678098 commented Feb 26, 2024

No description provided.

Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
@678098 678098 requested a review from a team as a code owner February 26, 2024 14:53
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
@678098 678098 changed the title [WIP]CI: MacOS build check CI: MacOS arm build check Feb 26, 2024
pniedzielski
pniedzielski previously approved these changes Feb 26, 2024
Copy link
Collaborator

@pniedzielski pniedzielski left a comment

Choose a reason for hiding this comment

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

I skimmed through the build logs and saw only expected warnings for an arm64 Mac build.

Depending on how long this takes, it make be worthwhile to cache build artifacts of BDE etc, since arm64 runners are more expensive.

bin/build-darwin.sh Outdated Show resolved Hide resolved
pip install -r ${{ github.workspace }}/src/python/requirements.txt
- name: Build BlazingMQ
run: bin/build-darwin.sh

Copy link
Collaborator

Choose a reason for hiding this comment

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

Does it make sense to run tests here (at least unit tests)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think UTs might fail on MacOS, and don't have time right now to fix it

Signed-off-by: Evgeny Malygin <emalygin@bloomberg.net>
@678098 678098 merged commit b83f8da into main Feb 26, 2024
8 checks passed
@678098 678098 deleted the 678098-patch-1 branch February 26, 2024 23:08
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.

None yet

2 participants