microsoft/Terminal

Passing a commandline to an elevate:true profile on the commandline doesn't separate args

Open

#12 597 ouverte le 28 févr. 2022

Voir sur GitHub
 (1 commentaire) (1 réaction) (0 assignés)C++ (3 212 forks)batch import
Area-CommandlineHelp WantedIssue-BugPriority-2Product-Terminal

Métriques du dépôt

Stars
 (35 764 stars)
Métriques de merge PR
 (Merge moyen 27j 19h) (24 PRs mergées en 30 j)

Description

(from a teams convo)

I bet that

wt --profile "Admin Cmd" -- cmd.exe /c D:\os\src\tools\razzle.cmd amd64fre developer_dir D:\Repos\developer\groups\jevansa no_opt

would work

[error 2147942402 (0x80070002) when launching `"cmd.exe /c D:\os\src\tools\razzle.cmd amd64fre developer_dir D:\Repos\developer\groups\jevansa no_opt"'] I ran: wt --profile "Admin Cmd" -- cmd.exe /c D:\os\src\tools\razzle.cmd amd64fre developer_dir D:\Repos\developer\groups\jevansa no_opt

I think it's still trying to take the whole string and pass to CreateProcess. Is there a way to get it split up?

it might be the fact that when you do have an elevate:true profile, we have to CreateProcess another terminal process, and I bet we're passing the whole bit after -- as a single arg to the elevated wt

Guide contributeur

Direction de recherche
Inspectez le chemin de code pour lancer un profil de terminal élevé, en vous concentrant sur la façon dont les arguments après ' ' sont passés à CreateProcess. Vérifiez s'ils sont concaténés en une seule chaîne au lieu d'être divisés en arguments séparés.
Stack technique
cpp
Domaine
desktopcli
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.
Plutôt claire
Prérequis
C++Windows API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30