Now that go1.5 has cgo support everywhere, we should just stop using sys calls and make this use tcsetattr and tcgetattr C functions. I've done this earlier today in a fork of termbox-go so that my port of it now works quite nicely on illumos (and pretty much everywhere else except for nacl) -- you can see part of this work here: https://github.com/nsf/termbox-go/pull/99
Hopefully termbox-go will accept my PR. It wouldn't be unreasonable to make a similar change to the ssh package. On Mon, Sep 21, 2015 at 1:14 PM, Alexander Kolbasov <[email protected]> wrote: > The only part that isn’t working is: > > $ github.com/mitchellh/cli > ~/go/src/github.com/mitchellh/cli/ui.go:79: undefined: terminal.IsTermina > ~/go/src/github.com/mitchellh/cli/ui.go:81: undefined: > terminal.ReadPassword > > These come from > > golang.org/x/crypto/ssh/terminal > > which compiles but apparently doesn’t define these two for Solaris. So the > actual fix should go to this package to support > > ./util_bsd.go:11: undefined: syscall.TIOCGETA > ./util_bsd.go:12: undefined: syscall.TIOCSETA > > This can probably be done within golang.org/x/sys/unix package which > should include some support for TIOCSETA/TIOCGETA and the > golang.org/x/crypto/ssh/terminal should be fixed to use this on Solaris. > > - Alex > *smartos-discuss* | Archives > <https://www.listbox.com/member/archive/184463/=now> > <https://www.listbox.com/member/archive/rss/184463/22103350-51080293> | > Modify > <https://www.listbox.com/member/?&;> > Your Subscription <http://www.listbox.com> > ------------------------------------------- smartos-discuss Archives: https://www.listbox.com/member/archive/184463/=now RSS Feed: https://www.listbox.com/member/archive/rss/184463/25769125-55cfbc00 Modify Your Subscription: https://www.listbox.com/member/?member_id=25769125&id_secret=25769125-7688e9fb Powered by Listbox: http://www.listbox.com
