Canop/broot

Username with one or more spaces causes an error

Fermée

#849 ouverte le 7 mars 2024

Voir sur GitHubGuide contributeur
 (9 commentaires) (1 réaction) (0 personne assignée)Rust (213 forks)batch import
bughelp wantedwindows

Métriques du dépôt

Stars
 (9 383 étoiles)
Métriques de merge PR
 (Merge moyen 53m) (1 PR mergée en 30 j)

Description

Steps to reproduce

  1. (optional) change your username to include one or more spaces
  2. If you have already br installed, run br --install
  3. run broot --install
  4. the command above creates $HOME\Documents\WindowsPowershell\Profile.ps1 with the following contents (when username is "User Name")
. C:\Users\User\ Name\AppData\Roaming\dystroy\broot\config\launcher\powershell\br.ps1

Possible solution

Use $HOME environment variable instead so windows will automatically expand it on execution appropriately.

Guide contributeur

Direction de recherche
Examinez le code de generation de profil PowerShell dans broot et remplacez le chemin code en dur par la variable d'environnement $HOME pour gerer les espaces dans les noms d'utilisateur.
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.
2
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
PowerShell
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70