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.

Sign up for GitHub

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

Jump to bottom

Python dependencies #8

Open
yochananmarqos opened this issue Mar 9, 2020 · 6 comments
Open

Python dependencies #8

yochananmarqos opened this issue Mar 9, 2020 · 6 comments
Labels
good first issue Good for newcomers

Comments

@yochananmarqos
Copy link

grpcio-tools should be a setup dependency, not a runtime dependency. In turn grpcio and protobuf should be runtime dependencies.
@gustavo-iniguez-goya
Copy link
Owner

Hello @yochananmarqos !

You're right, I'll update it soon. Thank you.

@yochananmarqos
Copy link
Author

Also configparser has been obsolete since Python 3.5.

gustavo-iniguez-goya added a commit that referenced this issue Mar 12, 2020
@gustavo-iniguez-goya
UI: removed configparser dependency
2d5b89a 
It looks like it's part of python since 3.2, and as far as I can tell
without the pip dependency work just fine (tested on ubuntu >=14.x, mint
>= 18, debian >= 8.x).

It should also help packaging for ArchLinux #8.
@aral
Copy link

aral commented Mar 19, 2020 •
edited

Assuming this is why I’m getting No module named 'grpc' when I try to run /usr/bin/opensnitch-ui from the .deb install?

(Thanks for working on this, by the way. Having a .deb install for this going to be great once it starts working as I never managed to get the original project to compile properly on my box.) :)

@gustavo-iniguez-goya
Copy link
Owner

the deb should work fine just out of the box (tested on clean debian >= 8, ubuntu >= 14.x), did you have an installation of the original repo?

Launch the GUI from command line and post the exception you get (or better, open a new issue since this one is particular to Arch). Thank you for the report by the way!

@metal450 metal450 mentioned this issue May 23, 2020
Closed
@gustavo-iniguez-goya gustavo-iniguez-goya added the good first issue Good for newcomers label Jul 28, 2020
@NP-Hardass
Copy link

opensnitch/ui/opensnitch/ui_pb2.py requires protobuf

@gustavo-iniguez-goya
Copy link
Owner

Thank you! I'll add it, and remove the constraint of pyqt 5.10.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@aral @gustavo-iniguez-goya @NP-Hardass @yochananmarqos