Skip to content

Cherry-pick "[lldb] Fix missing characters when autocompleting LLDB commands in REPL" to swift/master #1421

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
merged 1 commit into from
Jul 15, 2020

Conversation

poya
Copy link

@poya poya commented Jul 5, 2020

Cherry-pick of recent fix for https://bugs.swift.org/browse/SR-12867 in swift/master-next to swift/master

Differential Revision: https://reviews.llvm.org/D82835

@poya
Copy link
Author

poya commented Jul 5, 2020

@Teemperor Porting this recent fix to swift/master to be able to resolve the bug report as fixed. I'm not fully familiar with the branching strategy used, whether this fix should go into any other stable/release branch at this time?

@Teemperor
Copy link

The LLDB change goes into stable/ and the swift test goes into swift/master (it's a bit confusing). I did the cherry-pick here so you can just drop the LLDB change from this PR and we're all set!

@Teemperor
Copy link

(Also sorry for the delay)

@Teemperor
Copy link

@swift-ci test

@fredriss
Copy link

fredriss commented Jul 8, 2020

The test should also go to swift/master-next

@poya poya force-pushed the port/bug/SR-12867 branch from e7a3c4c to 1caa24e Compare July 9, 2020 01:14
@poya
Copy link
Author

poya commented Jul 9, 2020

@Teemperor LLDB change dropped, now only Swift REPL test in this PR

@poya
Copy link
Author

poya commented Jul 9, 2020

The test should also go to swift/master-next

@fredriss The test was originally added to swift/master-next in #1388 so that part has already been taken care of

@fredriss
Copy link

fredriss commented Jul 9, 2020

@swift-ci test

@poya
Copy link
Author

poya commented Jul 15, 2020

@Teemperor @fredriss Anything missing, or ready to merge?

@Teemperor
Copy link

This is only a test so I'll go ahead and merge this. The change for this already landed.

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