Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update status_shortcuts.rb #245

Closed
wants to merge 1 commit into from
Closed

Update status_shortcuts.rb #245

wants to merge 1 commit into from

Conversation

sarbull
Copy link

@sarbull sarbull commented Feb 8, 2018

References #242

@ghthor
Copy link
Member

ghthor commented Feb 14, 2018

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
Copy link
Author

sarbull commented Feb 14, 2018

@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
Copy link
Member

ghthor commented Feb 19, 2018

👍 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.

@ghthor ghthor closed this Aug 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants