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

switch to ergochat/readline #33305

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

slingamn
Copy link

@slingamn slingamn commented Jun 2, 2023

This switches the Terraform console library from chzyer/readline, which is unmaintained, to a fork ergochat/readline, maintained by me and @wader.

This will fix a number of bugs, including #33195. It may also make possible Solaris support for the console, which is currently compiled out:

"The readline library Terraform currently uses for the interactive\n" +
"console is not supported by Solaris. Interactive mode is therefore\n" +
"not supported on Solaris currently."))

I can confirm that our fork builds with GOOS=solaris, but I don't have a Solaris environment to confirm that the runtime behavior is as expected.

Fixes #33195

Target Release

1.5.x

Draft CHANGELOG entry

BUG FIXES

  • terraform console: Fixed crash from Ctrl-C when searching an empty history

@hashicorp-cla
Copy link

hashicorp-cla commented Jun 2, 2023

CLA assistant check
All committers have signed the CLA.

@crw
Copy link
Collaborator

crw commented Jun 2, 2023

Thanks for this submission! To set expectations, we will need to have this reviewed by the security team and we are in the beta/RC period for 1.5 so it may be some time before this is reviewed. Thanks again!

@slingamn
Copy link
Author

Just wanted to check in about this again?

@crw
Copy link
Collaborator

crw commented Jan 3, 2024

Thanks for the bump. I'll bring it up in triage again, to see if we want to pursue this in the near future.

@slingamn
Copy link
Author

(Updating to our latest release, v0.1.0)

@crw
Copy link
Collaborator

crw commented Jan 16, 2024

Thanks for this. We are going to work on a process for updating dependencies before new releases. I'll keep you posted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Panic in Terraform console command
3 participants