Skip to content

Replace usage of the atty crate with std::io::IsTerminal trait#1465

Merged
dandavison merged 1 commit into
dandavison:masterfrom
nickelc:no-atty
Jul 8, 2023
Merged

Replace usage of the atty crate with std::io::IsTerminal trait#1465
dandavison merged 1 commit into
dandavison:masterfrom
nickelc:no-atty

Conversation

@nickelc

@nickelc nickelc commented Jul 8, 2023

Copy link
Copy Markdown
Contributor

NOTE: delta will require Rust 1.70 (Release 2023-06-01) with the changes.

Rust 1.70 stabilized the IsTerminal trait and the newest version of grep-cli crate depending on it, makes it possible to remove the unmaintained atty crate from the build.

Crate:     atty
Version:   0.2.14
Warning:   unsound
Title:     Potential unaligned read
Date:      2021-07-04
ID:        RUSTSEC-2021-0145
URL:       https://rustsec.org/advisories/RUSTSEC-2021-0145

Rust 1.70 stabilized the `IsTerminal` trait and the newest version of
`grep-cli` crate depending on it, makes it possible to remove the
unmaintained `atty` crate from the build.

```
Crate:     atty
Version:   0.2.14
Warning:   unsound
Title:     Potential unaligned read
Date:      2021-07-04
ID:        RUSTSEC-2021-0145
URL:       https://rustsec.org/advisories/RUSTSEC-2021-0145
```
@dandavison

dandavison commented Jul 8, 2023

Copy link
Copy Markdown
Owner

Thanks @nickelc, LGTM. All the CI builds passed; can you think of anything related to release that might have a problem with the requirement for a recent Rust version?

@nickelc

nickelc commented Jul 8, 2023

Copy link
Copy Markdown
Contributor Author

I can only think of some distributions that don't provide the latest rust version yet.

@dandavison dandavison merged commit 915662f into dandavison:master Jul 8, 2023
@dandavison

Copy link
Copy Markdown
Owner

Right, thanks again.

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.

2 participants