Fix sparkup.py #60

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

kitmonisit commented Apr 1, 2012

I'm not sure why sparkup.py contained only 1 line:

sparkup

This must have been overlooked. This commit fixes that.

Fix sparkup.py
sparkup.py contained only 1 line: sparkup
Contributor

kitmonisit commented Apr 1, 2012

The sparkup.py fix kitmonisit/sparkup@cb6b023 is necessary for the out-of-the-box Pathogen installation.

Collaborator

faceleg commented Aug 22, 2013

Thanks for doing this, and sorry this has been left for so long!

How tough would it be to make this work for Bundle and NeoBundle as well (or will it already)?

@faceleg faceleg closed this Oct 1, 2013

Contributor

blueyed commented Oct 7, 2013

I think this pull request is a wrong fix (and will not work currently anymore, since $SPARKUP_PY points at nowhere after the latest commits). I think sparkup.py has been / was meant to be a symlink initially to sparkup?!

However, this does not appear to have been merged (which is good).

All it should take for a out-of-the box functionality is that you link the "vim" folder of sparkup into your "bundle" directory. In my "bundle" dir I have done "ln -s sparkup/vim sparkup-vim", and Vim will pick it up (all dirs in "bundle" get considered).

I will add instructions to the Vim README.

Contributor

blueyed commented Oct 7, 2013

This got lost due to git rebase --abort by accident.
A pull request to enhance it would be appreciated for sure.. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment