Skip to content

[build] Reenable lldb tests for macOS toolchain package #35357

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

Merged

Conversation

kastiglione
Copy link
Contributor

@kastiglione kastiglione commented Jan 11, 2021

Reenable lldb tests for package builds. The cause of the test failures have been fixed in the recent releases of lldb.

rdar://67820344

@kastiglione kastiglione requested a review from a team as a code owner January 11, 2021 23:04
@kastiglione
Copy link
Contributor Author

@shahmishal what's the right way to test this? Is it this?

preset=buildbot_osx_package
@ swift-ci test with preset macOS Platform

Or do I need to include "toolchain" too?

@kastiglione
Copy link
Contributor Author

@shahmishal friendly ping

@kastiglione
Copy link
Contributor Author

preset=buildbot_osx_package
@swift-ci test with preset macOS Platform

@shahmishal
Copy link
Member

@swift-ci build toolchain macOS

2 similar comments
@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione kastiglione force-pushed the build-Reenable-lldb-tests-for-macOS-toolchain-package branch from 890bafe to bb1f6ff Compare February 4, 2021 20:51
@kastiglione kastiglione changed the base branch from release/5.4 to main February 4, 2021 20:51
@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione kastiglione force-pushed the build-Reenable-lldb-tests-for-macOS-toolchain-package branch from bb1f6ff to 85698e2 Compare February 11, 2021 19:59
@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

7 similar comments
@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione
Copy link
Contributor Author

There are three failures, I have a fix for one. For the other two, I think we could disable them, in order to re-enable this test and prevent regressions to the package build test.

@kastiglione kastiglione force-pushed the build-Reenable-lldb-tests-for-macOS-toolchain-package branch from 85698e2 to 64f83f5 Compare April 29, 2023 21:02
@kastiglione
Copy link
Contributor Author

@swift-ci build toolchain macOS

@kastiglione
Copy link
Contributor Author

@swift-ci smoke test

@kastiglione kastiglione requested a review from adrian-prantl May 2, 2023 03:18
@kastiglione kastiglione merged commit 8f1b5c7 into main May 2, 2023
@kastiglione kastiglione deleted the build-Reenable-lldb-tests-for-macOS-toolchain-package branch May 2, 2023 22:32
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.

3 participants