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

Use golang.org/x/sys/unix for android and *BSD #49

Merged
merged 3 commits into from Jan 21, 2020

Conversation

tklauser
Copy link
Contributor

Re-use implementation in isatty_tcgets.go for android and implement isatty_bsd.go using IoctlGetTermios from golang.org/x/sys/unix. Also, clean up unused version hashes from go.mod.

See individual commits for details.

The implementation for android uses the same ioctl as linux and aix
which is already implemented in isatty_tcgets.go

Also, this no longer requires to use the frozen syscall package.
Use unix.IoctlGetTermios to implement IsTerminal on *BSD and no longer
requires to use the frozen syscall package.
Clean up unused version hashes in go.sum by running `go mod tidy`.
@codecov-io
Copy link

codecov-io commented Jan 21, 2020

Codecov Report

Merging #49 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #49   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines           5      5           
=====================================
  Hits            5      5
Impacted Files Coverage Δ
isatty_tcgets.go 100% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a28c40...6ede958. Read the comment docs.

@mattn mattn merged commit 7b513a9 into mattn:master Jan 21, 2020
@mattn
Copy link
Owner

mattn commented Jan 21, 2020

Thank you.

@tklauser tklauser deleted the use-x-sys branch January 22, 2020 08:45
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.

None yet

3 participants