Skip to content

Add lldb-dap to Swift distributions (#88482) #8573

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

adam-fowler
Copy link

This includes the lldb-dap executable in the MacOS and Linux distributions of Swift. Currently there is a commit in the Apple repo to do this for just MacOS #8176. This PR extends this to both Linux and MacOS and brings the change upstream.

(Cherry-pick of 2c2377d)

This includes the lldb-dap executable in the MacOS and Linux
distributions of Swift. Currently there is a commit in the Apple repo to
do this for just MacOS swiftlang#8176.
This PR extends this to both Linux and MacOS and brings the change
upstream.

@JDevlieghere @adrian-prantl
@adam-fowler
Copy link
Author

@JDevlieghere Is there any other branch this needs to be merged into to get it to be included in the main nightly and not just the 6.0 nightly?

@JDevlieghere
Copy link

@JDevlieghere Is there any other branch this needs to be merged into to get it to be included in the main nightly and not just the 6.0 nightly?

This PR covers both. We have an auto merger from the 6.0 (swift/release/6.0) to the stable branch (20230725) so it will automatically end up there.

@JDevlieghere
Copy link

@swift-ci test

@adam-fowler
Copy link
Author

@JDevlieghere any reason this hasn't been merged yet?

@JDevlieghere JDevlieghere merged commit 7b8e634 into swiftlang:swift/release/6.0 Apr 30, 2024
@JDevlieghere
Copy link

@JDevlieghere any reason this hasn't been merged yet?

No reason other than I didn't realize you were waiting on me :-)

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