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.

Sign up for GitHub

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

Jump to bottom

Replace abduco with dtach #446

Merged
merged 3 commits into from
Oct 5, 2023
Merged

Replace abduco with dtach #446

merged 3 commits into from
Oct 5, 2023

Conversation

michaeljguarino
Copy link
Member

Summary

Some users have seen issues with abduco's re-binding to existing sessions freezing screens. Dtach doesn't have history but also doesn't have that issue so trying it again.

Test Plan

test shells

Checklist

  • If required, I have updated the Plural documentation accordingly.
  • I have added tests to cover my changes.
  • I have added a meaningful title and summary to convey the impact of this PR to a user.
  • I have added relevant labels to this PR to help with categorization for release notes.

@michaeljguarino
Replace abduco with dtach
04c9227 
Some users have seen issues with abduco's re-binding to existing sessions freezing screens.  Dtach doesn't have history but also doesn't have that issue so trying it again.
@michaeljguarino michaeljguarino added the enhancement New feature or request label Sep 8, 2023
@michaeljguarino michaeljguarino merged commit e27431c into main Oct 5, 2023
11 checks passed
@michaeljguarino michaeljguarino deleted the replace-abduco-dtach branch October 5, 2023 14:39
michaeljguarino added a commit that referenced this pull request Aug 28, 2024
@michaeljguarino
Replace abduco with dtach (#446)
2ed4087 
* Replace abduco with dtach

Some users have seen issues with abduco's re-binding to existing sessions freezing screens.  Dtach doesn't have history but also doesn't have that issue so trying it again.

* Revert "Replace abduco with dtach"

This reverts commit 04c9227.

* use dtach again
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maciaszczykm maciaszczykm maciaszczykm approved these changes

@davidspek davidspek Awaiting requested review from davidspek

@zreigz zreigz Awaiting requested review from zreigz zreigz is a code owner

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@michaeljguarino @maciaszczykm