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

Added flexibility to get.vimfiles.sh #2

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

Conversation

ccalvert
Copy link

@ccalvert ccalvert commented Oct 6, 2011

Jim,

I modified get.vimfiles.sh to use a slightly different argument to "git clone". This allows it to work correctly on systems that have multiple github identities.

Note that you should cherry pick the changes to get.vimfiles.sh. The change to vimrc_settings was just to update my color scheme.

Charles

@coderifous
Copy link
Owner

Hey Charles - I'd like to go the other direction with this file. Instead of having the URL broken out into variables - it would be better just to assign the URL as a string without any variable construction at all. A chat w/ Ryan convinced me that having variables to build up a URL in this context is overkill - even just the repo-owner name like I had before.

If you give me a commit that squashes it down to just a single line for assigning the URL variable, I will merge it in.

-Jim

Charles Calvert added 2 commits October 25, 2011 23:24
…eliminate the separate REPO_NAME variable, because its needed elsewhere. Added ability to back up existing .vim, .vimrc and .gvimrc files before creating symbolic links.
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