zsh-users/zsh-syntax-highlighting

PATH_DIRS shouldn't be honoured after `sudo`

Open

#595 aperta il 24 dic 2018

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Shell (1371 fork)batch import
Buggood first issuesyntax:command-word

Metriche repository

Star
 (22.617 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

The main highlighter explicitly sets PATH_DIRS if the user has that enabled:

https://github.com/zsh-users/zsh-syntax-highlighting/blob/55f846c673661bde03ef97333be7889f54079c7b/highlighters/main/main-highlighter.zsh#L316-L318

This is so, if $^path/foo/bar(*N) exists, % foo/bar will be highlighted in green. However, that also happens in sudo foo/bar, even though sudo doesn't do path searches when there's a / in the argument — at least, I think that's how sudo behaves, but double-check that before making any code change on our end.

Guida contributor

Direzione di ricerca
Indaga il comportamento di sudo: conferma se sudo esegue ricerche nel percorso quando l'argomento contiene una barra (probabilmente no). Quindi modifica main highlighter.zsh intorno alle righe 316-318 per saltare l'impostazione di PATH DIRS per comandi come sudo. La correzione è localizzata e dovrebbe essere semplice.
Tech stack
shell
Dominio
developer experience
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
GitZsh
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70