Skip to content

ci: Adds MacOS Bazel Builder#9569

Open
QuantamHD wants to merge 5 commits intoThe-OpenROAD-Project:masterfrom
QuantamHD:macos-bazel
Open

ci: Adds MacOS Bazel Builder#9569
QuantamHD wants to merge 5 commits intoThe-OpenROAD-Project:masterfrom
QuantamHD:macos-bazel

Conversation

@QuantamHD
Copy link
Collaborator

No description provided.

Signed-off-by: Ethan Mahintorabi <ethanmoon@google.com>
@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Signed-off-by: Ethan Mahintorabi <ethanmoon@google.com>
Signed-off-by: Ethan Mahintorabi <ethanmoon@google.com>
@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment
@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"


jobs:
Mac-Build:
if: github.repository_owner != 'The-OpenROAD-Project' && github.repository_owner != 'The-OpenROAD-Project-private' && github.repository_owner != 'The-OpenROAD-Project-staging'
Copy link
Member

Choose a reason for hiding this comment

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

What is the goal of excluding these?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This mirrors the other macos action

image

@maliberty
Copy link
Member

Introduced in e3ee598 was for back when we had a self-hosted mac build. We may again someday but this currently means the action never runs.

Signed-off-by: Ethan Mahintorabi <ethanmoon@google.com>
@QuantamHD
Copy link
Collaborator Author

Fixed

@QuantamHD
Copy link
Collaborator Author

As you point out we should probably populate a bazel macos cache for the CI. Right now we're limited by the 10GB limit action cache.

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Signed-off-by: Ethan Mahintorabi <ethanmoon@google.com>
@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@QuantamHD
Copy link
Collaborator Author

@maliberty the ci failed pulling a docker container should be good to go

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