search

aniketpanjwani / chomper

Internet blocker for the Linux desktop.
https://addictedto.tech/chomper/
GNU General Public License v3.0
356 stars 43 forks source link

Add /sbin:/usr/sbin to PATH to make sure iptables is found. #23

Closed sarg closed 6 years ago

sarg commented 6 years ago

Fixes #22

aniketpanjwani commented 6 years ago

This looks good, but for the same reason, we will need to adjust the reset task in the Makefile. I.e. we need to turn this:

reset:
    sudo env PATH=$(PATH) ${INTERPRETER}3.6m ${CURRENT_DIR}/chomper/reset.py

into this:

reset:
    sudo env PATH=$(PATH):/sbin/usr/sbin ${INTERPRETER}3.6m ${CURRENT_DIR}/chomper/reset.py

Could you also add that to the pull request? Then, I'm happy to merge it in.

aniketpanjwani commented 6 years ago

Awesome, thanks Sergey!