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.

Sign up for GitHub

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

Jump to bottom

Install/update ignores changed remotes #48

Closed
dag opened this issue May 29, 2011 · 7 comments
Closed

Install/update ignores changed remotes #48

dag opened this issue May 29, 2011 · 7 comments
Assignees
@gmarik
Labels
bug
Milestone
Vundle 1

Comments

@dag
Copy link
Contributor

dag commented May 29, 2011

If I fork a bundle and change my configuration to use my fork, neither :BundleInstall or :BundleInstall! will pick it up - I would guess Vundle is just doing git pull without bothering about remotes.

The workaround of course is to delete the existing clone (or manually change the remotes) but it would be better if Vundle handled it for us.
@gmarik
Copy link
Contributor

gmarik commented May 29, 2011

+1 on this.
Changing remotes sounds like a cleanest solution...

@gmarik
Copy link
Contributor

gmarik commented Jul 24, 2011

Hey @dag.
pushed v branch which addresses this issue.
If you could find some time to test it would be awesome!
Thanks!

$ cd vundle && git pull && git checkout -b v origin/v

@gmarik gmarik mentioned this issue Dec 18, 2011
Closed
@ghost ghost assigned gmarik Dec 20, 2011
@mutewinter
Copy link

+1 to this, run into it all the time.

@Valloric
Copy link

Valloric commented Feb 1, 2013

I run into this all the time as well. Any chance of this getting fixed any time soon?

@vheon
Copy link
Contributor

vheon commented Aug 23, 2013

🆙 Any news?

@MarcWeber
Copy link

related to #367

@jdevera jdevera mentioned this issue Apr 4, 2014
Closed
@jdevera jdevera removed the feature label Apr 4, 2014
jdevera added a commit that referenced this issue Apr 15, 2014
@jdevera
Treat changed remotes as new plugins
f5a208b 
If one had this plugin:

Plugin 'foo/bar'

And now switches to another one with the same name:

Plugin 'baz/bar'

Recognise this scenario and replace the old plugin with the new one,
rather than silently assuming they are the same.

Fixes #367 #48 #183 #400
@jdevera jdevera mentioned this issue Apr 15, 2014
Merged
jdevera added a commit that referenced this issue Apr 15, 2014
@jdevera
Treat changed remotes as new plugins
53cb121 
If one had this plugin:

Plugin 'foo/bar'

And now switches to another one with the same name:

Plugin 'baz/bar'

Recognise this scenario and replace the old plugin with the new one,
rather than silently assuming they are the same.

Fixes #367 #48 #183 #400
@jdevera
Copy link
Contributor

jdevera commented Apr 23, 2014

Fixed in f5a208b

@jdevera jdevera closed this as completed Apr 23, 2014
@ReadmeCritic ReadmeCritic mentioned this issue Nov 18, 2015
Merged
homu added a commit to ycm-core/YouCompleteMe that referenced this issue Nov 21, 2015
@homu
Auto merge of #1786 - ReadmeCritic:master, r=Valloric
96f0d94 
Update README URLs based on HTTP redirects

Created with https://github.com/dkhamsing/frankenstein

### GitHub Corrected URLs
Was | Now
--- | ---
http://mxcl.github.com/homebrew/ | http://mxcl.github.io/homebrew/
VundleVim/Vundle.vim#48 | VundleVim/Vundle.vim#48
https://github.com/nosami/OmniSharpServer | https://github.com/OmniSharp/omnisharp-server

### Other Corrected URLs
Was | Now
--- | ---
http://en.wikipedia.org/wiki/Subsequence | https://en.wikipedia.org/wiki/Subsequence
http://www.cmake.org/cmake/resources/software.html | https://cmake.org/download/

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/valloric/youcompleteme/1786)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gmarik gmarik

Labels
bug
Projects
None yet
Milestone
Vundle 1
Development

No branches or pull requests
7 participants
@dag @gmarik @MarcWeber @jdevera @vheon @mutewinter @Valloric