search

scmbreeze / scm_breeze

Adds numbered shortcuts to the output git status, and much more
https://madebynathan.com/2011/10/19/git-shortcuts-like-youve-never-seen-before/
MIT License
2.82k stars 192 forks source link

Fix shell wrapping for exes with space in path. #299

Closed jameskeane closed 2 years ago

jameskeane commented 4 years ago

I experienced this using VS Code in WSL, the code CLI is in the Windows PATH that has a space in my user name.

Fixes #294.

Stephen2 commented 2 years ago

Any maintainers available to look at this PR and potentially merge?

scm_breeze unfortunately not useable on WSL2 easily, because of this

ghthor commented 2 years ago

I believe I looked at this and it broke some other stuff. But I'll take another look and see what I can do.

ghthor commented 2 years ago

Alright, I've been running with this change for a few days and haven't noticed any issues.

Stephen2 commented 2 years ago

Thank you all!