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

Fix git ahead/behind count on zsh #31

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rojspencer
Copy link

@{upstream}...HEAD needs to be quoted else zsh eats it.

@{upstream}...HEAD needs to be quoted else zsh eats it.
@rojspencer rojspencer closed this Mar 22, 2015
@rojspencer rojspencer reopened this Mar 22, 2015
[ ! $only_if_ssh ] only tests if the length of the string is non-zero.
So setting it to 0 or 1 resulted in it always being true.

Switch to a more standard boolean check, pulled from:
http://stackoverflow.com/questions/2953646/how-to-declare-and-use-boolean-variables-in-shell-script
@rojspencer
Copy link
Author

Apparently I don't understand how pull requests work on github.

Made a second fix for the only_if_ssh check on the host slice and somehow it became a part of this pull request. Should be pulled also but not sure how it automatically became part of this PR.

@edkolev
Copy link
Owner

edkolev commented Mar 26, 2015

Hm.. I can't reproduce. Which version of zsh are you using?

@rojspencer
Copy link
Author

Installed via homebrew.
zsh 5.0.7 (x86_64-apple-darwin14.0.0)
On OS X Yosemite

Also saw the same on a Fedora 20 Workstation with zsh version:
zsh 5.0.7 (x86_64-redhat-linux-gnu)

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