-
Notifications
You must be signed in to change notification settings - Fork 64
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
Errors using agnoster if svn isn't present #3
Comments
Agreed. Could you submit a PR? |
@bpinto Possibly. I'll have to request approval. |
I fixed this issue doing
|
@FabioAntunes Nice! Could you merge both if scenarios with an |
@bpinto sure, here's the merge function:
|
Hey, would you submit as a PR? Sorry for the trouble! |
Supressed warning when svn isn't installed. Fixes #3
I'm sorry, but this PR creates the bug documented in PR #5 - it shows the svn branch symbol at any time (on OS X). |
It appears that agnoster is looking to see if svn is installed but is causing an error in the process. Perhaps
which svn
might be a better match for this check?System: Ubuntu 14.04 x64
I fixed the issue by installing subversion
sudo apt-get install subversion
but I feel like users shouldn't have to do that.The text was updated successfully, but these errors were encountered: