LS job fails on windows vim if &shell is powershell
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Tranquilla
- Stack tecnologico
- node.js, powershell, vim
- Ambito
- operating-systems, tooling
Direzione di ricerca
Inizia da autoload/copilot/client.vim, nello specifico da copilot#client#New(), e verifica come viene preparato il comando quando Vim viene eseguito su Windows con &shell impostato su PowerShell o pwsh. Riproduci l’avvio del server di linguaggio in quell’ambiente e verifica che continui a funzionare anche con la configurazione della shell esistente; il lavoro è completo quando il server si avvia correttamente senza compromettere le altre piattaforme.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The language server runs in node.js and is launched using:
> npx @github/copilot-language-server@^1.408.0 --stdio
the problem is that the plugin (see copilot.vim/autoload/copilot/client.vim function copilot#client#New())
uses:
job_start(['npx', '@github/copilot-language-server@^1.408.0', '--stdio'], ...)
which fails (is a win32 CreateProcess(...) issue) and should instead use:
job_start('npx "@github/copilot-language-server@^1.408.0" --stdio', ...)
The following patch works for me:
diff --git a/autoload/copilot/client.vim b/autoload/copilot/client.vim
index c3ec862..c052bc8 100644
--- a/autoload/copilot/client.vim
+++ b/autoload/copilot/client.vim
@@ -736,7 +736,7 @@ function! copilot#client#New() abort
let instance.workspaceFolders[folder.uri] = v:true
endfor
call copilot#logger#Debug('Spawning ' . join(command, ' '))
- let is_win_shell = has('win32') && &shellcmdflag !~# '^-'
+ let is_win_shell = has('win32') && ( &shell =~? 'powershell\|pwsh' || &shellcmdflag !~# '^-')
if is_win_shell && command[0] !~# '[\/]'
let exepath = exepath(command[0])
if exepath !~? '\.exe$\|^$'
- Lingua principale
- Vim Script
- Stelle
- 11.7k
- Fork
- 812
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di github/copilot.vim
-
No link to DeepWiki Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
github/copilot.vim#303 ·
-
Neomarley Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 1/100
github/copilot.vim#304 ·
-
Readme. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 20/100
github/copilot.vim#302 · 4 commenti ·
-
Micky Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 1/100
github/copilot.vim#301 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 5/100
github/copilot.vim#300 ·
Tutte le issue di github/copilot.vim
Issue simili
-
libc
Difficoltà 2/5 1-2 giorni Idoneità per principianti 76/100
llvm/llvm-project#225506 · 1 commento ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
ValveSoftware/Proton#10167 ·
-
next-devel: s390x build fails — chccwdev/vmur/zkey missing from initramfs after s390utils 2.44 split Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
coreos/fedora-coreos-tracker#2228 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
pop-os/plymouth-theme#43 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
danielmiessler/LifeOS#2223 · 2 commenti ·