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

Plugins not available when installing via pip #73

Closed
benfishbus opened this issue Jul 16, 2024 · 6 comments
Closed

Plugins not available when installing via pip #73

benfishbus opened this issue Jul 16, 2024 · 6 comments

Comments

@benfishbus
Copy link

benfishbus commented Jul 16, 2024

SUMMARY

Many (all?) plugins give an error when Dool is installed using pip.

ISSUE TYPE
  • Bug Report
DOOL VERSION

1.3.2

OS / ENVIRONMENT

Platform posix/linux
Kernel 6.1.79-Unraid
Python 3.9.10 (main, Jan 16 2022, 02:41:55)
[GCC 11.2.0]

STEPS TO REPRODUCE
pip install dool
dool --top-io
EXPECTED RESULTS

Start the top-io plugin.

ACTUAL RESULTS

dool: option --top-io not recognized, try dool -h for a list of all the options

@benfishbus benfishbus changed the title Plugins not available Plugins not available when installing via pip Jul 16, 2024
@scottchiefbaker
Copy link
Owner

Dool checks for plugins in:

    os.path.expanduser('~/.dool/'),                              # home + /.dool/
    os.path.abspath(os.path.dirname(sys.argv[0])) + '/plugins/', # binary path + /plugins/
    '/usr/share/dool/',
    '/usr/local/share/dool/',

Do you see any files in any of those locations? Running dool --version will list all the plugins it finds, and what directory they're located in.

@raylu
Copy link
Contributor

raylu commented Jul 16, 2024

oh I knew about this when I set up the packaging. see #50

the main problem, though, is plugins/. it can't find any after installing

I wrote some more about what it would take to make the plugins installable and importable, but after

The plugins are nice, but not required.

I decided to forget about it

@scottchiefbaker
Copy link
Owner

@raylu is this something we want to include in the next version that gets published? I don't know how complicated it would be.

@raylu raylu mentioned this issue Jul 19, 2024
@raylu
Copy link
Contributor

raylu commented Jul 19, 2024

@scottchiefbaker
Copy link
Owner

@benfishbus is this issue resolved?

@benfishbus
Copy link
Author

Had to educate myself on installing stuff from test. It is working for me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants