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

Enable C/C++ remote tests on macOS #23085

Closed
wants to merge 2 commits into from

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Jul 24, 2024

Also ensure that the remote worker doesn't fail executions when the spawn fails, but instead sends back the result of the unsuccessful execution.

@fmeum
Copy link
Collaborator Author

fmeum commented Jul 24, 2024

FYI @tjgq I'm working on this now, but it will require further changes (e.g. #23089).

Support for Xcode env vars is aded to the remote worker, making it possible to run these tests.
@fmeum fmeum marked this pull request as ready for review September 4, 2024 11:37
@fmeum fmeum requested a review from a team as a code owner September 4, 2024 11:37
@github-actions github-actions bot added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Sep 4, 2024
@fmeum
Copy link
Collaborator Author

fmeum commented Sep 4, 2024

FYI @keith

@fmeum
Copy link
Collaborator Author

fmeum commented Sep 11, 2024

@tjgq I have been able to drop the apple_support patch thanks to a release containing the fix, this is now ready for review.

@fmeum
Copy link
Collaborator Author

fmeum commented Oct 1, 2024

@coeuvre Could you help getting this merged? It just adds test coverage.

Copy link
Member

@coeuvre coeuvre left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

@copybara-service copybara-service bot closed this in 509f79f Oct 1, 2024
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Oct 1, 2024
@fmeum fmeum deleted the macos-remote-tests branch October 1, 2024 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Remote-Exec Issues and PRs for the Execution (Remote) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants