zsh-users/zsh-syntax-highlighting

PATH_DIRS shouldn't be honoured after `sudo`

Offen

#595 geöffnet am 24.12.2018

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Shell (1.371 Forks)batch import
Buggood first issuesyntax:command-word

Repository-Metriken

Stars
 (22.617 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche das sudo Verhalten: Bestätige, ob sudo Pfadsuchen durchführt, wenn das Argument einen Schrägstrich enthält (wahrscheinlich nicht). Ändere dann main highlighter.zsh um die Zeilen 316-318 herum, um die Einstellung von PATH DIRS für Befehle wie sudo zu überspringen. Die Korrektur ist lokalisiert und sollte unkompliziert sein.
Tech Stack
shell
Domain
developer experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitZsh
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70