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

Update status_shortcuts.rb #245

Closed sarbull closed 5 years ago

sarbull commented 6 years ago

References #242

ghthor commented 6 years ago

Alright, if this fixes #242 then the issue has nothing to do with the ruby version, it is that /dev/null does not exist on WIndows. Rerouting the stderr output needs a crossplatform approach. What type of env are you running scm_breeze within on windows? You must not be using bash inside cygwin or msysgit, as I'd expect both those env to represent /dev/null.

Good news is that I'll soon have a native windows env personally so I can test these types of issues!

sarbull commented 6 years ago

@ghthor I think this might not really fix the initial issue, i found some sort of bugs. It seems to be working only when you run commands in the root folder of the project near .git folder.

If you try running git aliased commands in other folder it throws something other issue in 40% of the cases, i don't know why..

So theoretically this fixes it partially. Can you please test this? I am using this on a Windows 7 with Git Bash (cygwin, i think).

ghthor commented 6 years ago

:thumbsup: I will do a full round a testing and fixes when I purchase a new personal VR machine, which will be the at end of March 2018.