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

[bug] Missing requirement "rich" #2

Merged
merged 2 commits into from
Jun 3, 2024
Merged

[bug] Missing requirement "rich" #2

merged 2 commits into from
Jun 3, 2024

Conversation

lefayjey
Copy link
Contributor

@lefayjey lefayjey commented Jun 3, 2024

Hey!

Thanks again for these awesome tools. I had an error message for a missing requirement when installing it using pipx.

┌──(kali㉿kali)-[~]
└─$ pipx install git+https://github.com/p0dalirius/smbclient-ng --force
  installed package smbclientng 1.2, installed using Python 3.11.8
  These apps are now globally available
    - smbclientng
    - smbng
done! ✨ 🌟 ✨
                                                                                                                                                                                                                                            
┌──(kali㉿kali)-[~]
└─$ smbclientng
Traceback (most recent call last):
  File "/home/kali/.local/bin/smbclientng", line 5, in <module>
    from smbclientng.__main__ import main
  File "/home/kali/.local/pipx/venvs/smbclientng/lib/python3.11/site-packages/smbclientng/__main__.py", line 10, in <module>
    from smbclientng.core.InteractiveShell import InteractiveShell
  File "/home/kali/.local/pipx/venvs/smbclientng/lib/python3.11/site-packages/smbclientng/core/InteractiveShell.py", line 17, in <module>
    from rich.console import Console
ModuleNotFoundError: No module named 'rich'

The changes fixed it for me.
Thanks!

@p0dalirius p0dalirius self-assigned this Jun 3, 2024
@p0dalirius p0dalirius added the bug Something isn't working label Jun 3, 2024
@p0dalirius p0dalirius changed the title Missing requirement "rich" [bug] Missing requirement "rich" Jun 3, 2024
@p0dalirius p0dalirius merged commit 3dadb46 into p0dalirius:main Jun 3, 2024
2 checks passed
@p0dalirius
Copy link
Owner

Thank you for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants