gustavo-iniguez-goya / opensnitch

OpenSnitch is a GNU/Linux application firewall

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

make asks for sudo but does not explain why

licaon-kter opened this issue · comments

Debian Sid, amd64

Following https://github.com/gustavo-iniguez-goya/opensnitch/wiki/Compilation I've done:

apt-get install git golang libnetfilter-queue-dev libpcap-dev protobuf-compiler python3-pip
go get github.com/golang/protobuf/protoc-gen-go
python3 -m pip install --user grpcio-tools
go get github.com/gustavo-iniguez-goya/opensnitch
cd $GOPATH/src/github.com/gustavo-iniguez-goya/opensnitch

But now, make does this:

make[1]: Entering directory 'gopath/src/github.com/gustavo-iniguez-goya/opensnitch/proto'
python3 -m grpc_tools.protoc -I. --python_out=../ui/opensnitch/ --grpc_python_out=../ui/opensnitch/ ui.proto
make[1]: Leaving directory 'gopath/src/github.com/gustavo-iniguez-goya/opensnitch/proto'
make[1]: Entering directory 'gopath/src/github.com/gustavo-iniguez-goya/opensnitch/ui'
[sudo] password for localuser:

Are the Wiki steps incomplete?

/LE: so I guess the UI here https://github.com/gustavo-iniguez-goya/opensnitch/blob/main/Makefile#L13 wants some deps here: https://github.com/gustavo-iniguez-goya/opensnitch/blob/main/ui/Makefile#L10 which is explained in the Wiki post, but way below the main install block.

Should I modify the Wiki to add this info the in main block?

cd ui
sudo pip3 install -r requirements.txt

Hey @licaon-kter

Are the Wiki steps incomplete?

Yes, I've added pyqt5-dev-tools to the list of packages to install. It's needed for pyrcc5 command.

I think the best should be to remove sudo from ui/Makefile , and document on the wiki to launch it with sudo.

I think the best should be to remove sudo from ui/Makefile , and document on the wiki to launch it with sudo.

Yes, the current one just expects sudo which might not be available and setup