dtolnay/cargo-expand

Show progress bar

Ouverte

#62 ouverte le 20 avr. 2019

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)Rust (73 forks)github user discovery
help wanted

Métriques du dépôt

Stars
 (3 090 étoiles)
Métriques de merge PR
 (Merge moyen 4m) (1 PR mergée en 30 j)

Description

Currently the Cargo subprocess spawned by cargo-expand does not display Cargo's usual progress bar. I believe this is because we pipe the child's stderr to perform filtering here:

https://github.com/dtolnay/cargo-expand/blob/3ae4ca126a4bd557ef7f7954911d65db56525b37/src/main.rs#L358

and that means Cargo does not get a winsize from this ioctl:

https://github.com/rust-lang/cargo/blob/6be12653dcefb46ee7b605f063ee75b5e6cba513/src/cargo/core/shell.rs#L377

I tried a naive fix in #61 but it didn't work, the second ioctl fails with:

Inappropriate ioctl for device (os error 25)

Will need to figure out what's the correct way to provide a winsize to the child process.

Guide contributeur

Direction de recherche
Recherchez comment fournir un winsize au processus enfant, éventuellement en définissant la taille du terminal via des variables d’environnement ou en utilisant un pseudo terminal (pty) pour transférer le winsize.
Stack technique
rust
Domaine
cli
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustCargo
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40