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

Plugin breaks when multiple instances of same plugin found #870

Closed
asfaltboy opened this issue Oct 13, 2016 · 7 comments
Closed

Plugin breaks when multiple instances of same plugin found #870

asfaltboy opened this issue Oct 13, 2016 · 7 comments

Comments

@asfaltboy
Copy link
Contributor

We should probably have a detection precedence when searching for plugin to load:

  • local plugin dir
  • custom plugin dir
  • github maintained plugin dir
@sijis
Copy link
Contributor

sijis commented Oct 15, 2016

What order does it match now?
Also which errbot version?

@asfaltboy
Copy link
Contributor Author

Honestly, I have no clue! Yapsy is a weird creature, and I couldn't follow the code to understand how errbot's BotPluginManager really loads all the plugins. Regardless, it's less about the order and more that it might count the same plugin in different locations multiple times (it should only allow a unique plugin to be loaded once).

@sijis
Copy link
Contributor

sijis commented Oct 16, 2016

Gotcha. To be clear, the issue is that the same plugin is found in multiple locations (and loaded), so your suggestion is to find it (via a precedence order) and load it once.

@asfaltboy
Copy link
Contributor Author

Correct, at least as far as I understand the cause

On Sun, Oct 16, 2016, 19:09 Sijis Aviles notifications@github.com wrote:

Gotcha. To be clear, the issue is that the same plugin is found in
multiple locations (and loaded), so your suggestion is to find it (via a
precedence order) and load it once.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#870 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATn8DMl5tL8Hem8Abwv7xJMbenetj4fks5q0lpcgaJpZM4KV1HN
.

@sijis
Copy link
Contributor

sijis commented Oct 16, 2016

Last thing. Python and errbot version

@asfaltboy
Copy link
Contributor Author

Python 3.5.1, errbot version 9.9.9 (checked out from git, commit f2af0d7)

@sijis
Copy link
Contributor

sijis commented Jul 23, 2021

yapsy is no longer managing plugins, so its likely this issue has been resolved.
If not, a new issue can be opened with newer data.

@sijis sijis closed this as completed Jul 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants