Support linux systems that don't keep stty in /bin

[cameronfyfe]

Summary

System calls to stty to enable/disable terminal echos rely on a hardcoded path at /bin/stty.

This is due to hardcoded path strings in https://github.com/bgentry/speakeasy/tree/v0.1.0.

There is currently an issue and PR that should fix the issue once merged. I bumped the issue today.

Problem Definition

This prevents binaries using this library from being operable on systems that don't have stty located in /bin, unless the user moves stty or symlinks /bin/stty to point to the location on their system.

Proposal

• What
  • Use the system's search path to find stty instead of relying on hardcoded /bin/stty.
• How
  • Get this pull request pushed through and update dependency: Fix #22 - search for stty in PATH bgentry/speakeasy#24
  • or
  • Remove speakeasy and implement password entry directly or with other library that doesn't have this issue.

This would expand out-of-the box support for cosmos-sdk onto more system environments. In my specific case, this would allow me to run cosmos-sdk based binaries on NixOS systems.

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned

[alexanderbez]

It seems like the easiest approach is to just wait for that PR to be merged and released. Dependabot will automatically open a PR for the SDK to bump the version.

[julienrbrt]

For what it's worth, I have approved the PR there (ping @bgentry)
Closing this as when it gets implemented dependabot will alert us.