dtolnay/cargo-expand

Show progress bar

Aperta

#62 aperta il 20 apr 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Rust (73 fork)github user discovery
help wanted

Metriche repository

Star
 (3090 stelle)
Metriche merge PR
 (Merge medio 4m) (1 PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga su come fornire un winsize al processo figlio, possibilmente impostando la dimensione del terminale tramite variabili d'ambiente o utilizzando un pseudo terminale (pty) per inoltrare il winsize.
Tech stack
rust
Dominio
cli
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
RustCargo
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40