errbotio / err-backend-slackv3

Slack Events and RTM backend for Errbot
GNU General Public License v3.0
27 stars 28 forks source link

errbot-backend-slackv3 module install via pip inaccessible without specifying BOT_EXTRA_BACKEND_DIR #91

Closed p1co closed 1 year ago

p1co commented 2 years ago

Without specifying BOT_EXTRA_BACKEND_DIR, errbot does not know about errbot-backend-slackv3 when it is installed via pip.

Specifying something like BOT_EXTRA_BACKEND_DIR="/home/pico/err_test/lib/python3.10/site-packages" seems a bit ugly and unnecessary. Why not make it so site-packages are checked?

~/err_test » pip show errbot-backend-slackv3
Name: errbot-backend-slackv3 Version: 0.2.1 Summary: Errbot SlackV3 backend plugin Home-page: UNKNOWN Author: Errbot Author-email: License: UNKNOWN Location: /home/pico/err_test/lib/python3.10/site-packages Requires: aiohttp, markdown, slack-sdk, slackeventsapi Required-by: (err_test) --------------------------------------------------------------------------------------------------------------------------------------------------------------- ~/err_test » errbot
10:42:59 INFO errbot.bootstrap Found Storage plugin: Shelf. Traceback (most recent call last): File "/home/pico/err_test/bin/errbot", line 8, in sys.exit(main()) File "/home/pico/err_test/lib/python3.10/site-packages/errbot/cli.py", line 396, in main bootstrap(backend, root_logger, config, restore) File "/home/pico/err_test/lib/python3.10/site-packages/errbot/bootstrap.py", line 265, in bootstrap bot = setup_bot(bot_class, logger, config, restore) File "/home/pico/err_test/lib/python3.10/site-packages/errbot/bootstrap.py", line 172, in setup_bot backendpm = BackendPluginManager( File "/home/pico/err_test/lib/python3.10/site-packages/errbot/backend_plugin_manager.py", line 53, in init raise PluginNotFoundException( errbot.backend_plugin_manager.PluginNotFoundException: Could not find the plugin named SlackV3 in ['/home/pico/err_test/lib/python3.10/site-packages/errbot/backends']. (err_test) ---------------------------------------------------------------------------------------------------------------------------------------------------------------

nzlosh commented 2 years ago

A fix was pushed to errbot master branch that should address this issue.