IJHack / QtPass / 27726134529

55%

push

github

fix(#1509): replace broken WSL wslpath shell substitution with real wslpath call (#1569)

* fix(#1509): replace broken WSL \$(wslpath) shell substitution with real wslpath call

pgit(), pgpg() and grepMatchFile() constructed "$(wslpath /path)" strings and
passed them as QProcess arguments. QProcess does not invoke a shell, so the
substitution never expanded — WSL-mode git/gpg received the literal string as
an argument and failed.

Fix: call `wsl wslpath <path>` synchronously via Executor::executeBlocking and
use the captured output as the translated path. Non-WSL paths are unchanged.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(#1509): fall back to normalizedPath when wsl wslpath fails

All three call sites (pgit, pgpg, grepMatchFile) discarded the return
value of executeBlocking; a non-zero exit or a WSL process failure left
wslPath as an empty string, which was then passed to git/gpg causing
confusing empty-path errors.

Check both the exit code and the trimmed output; fall back to the
original normalizedPath / filePath when either is unsatisfactory.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

0 of 13 new or added lines in 1 file covered. (0.0%)

3666 of 6673 relevant lines covered (54.94%)

29.48 hits per line