Changed install_plugins() to allow remote installation without local plugins #3

merged 2 commits into from Mar 23, 2014

2 participants


Line 953 in install_plugins() previously returned 0 if there were no local plugins to remotely install. In 3.07, this had no effect, but in 3.08 this causes the chain of functions performing the remote installation to fail prematurely without remotely installing aliases.

Since bcvi can operate without plugins, this patch allows users without local bcvi plugins to perform remote installations.

sapid added some commits Mar 23, 2014
@sapid sapid install_plugins() now permits no local plugins
install_plugins() will now let you finish remote installation even if bcvi has no local plugins.
@sapid sapid Recording changes to bcvi's install_plugins() 90a84b6
@grantm grantm merged commit 553f4dd into grantm:master Mar 23, 2014

Good catch - thanks for the contribution.

@grantm grantm added a commit that referenced this pull request Mar 24, 2014
@grantm v3.09
        - fixes #3: allow remote installation without local plugins
          (pull request from Will Crawford)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment