kboghdady / youTube_ads_4_pi-hole

YouTube script to add the new Ads list for Pi-hole
2.45k stars 266 forks source link

sqlite3 and nice #133

Open arfazero opened 1 year ago

arfazero commented 1 year ago

the codes a bit jank (i like jank) for the new pihole and you need to install sqlite3 as it looks to be a prerequisite, im also looking at adding a cpu restriction so it can run without completely locking up the pihole and slowing down dns requests,

maybe something like

0 /1 sudo nice -15 /home/pi/youTube_ads_4_pi-hole/youtube.sh >/dev/null

it does seem to do what it says with the YouTube adds, just needs a little more polish and an updated readme would also label as wip

9/10 ign :D nice work

devhl-labs commented 1 year ago

I also see sqlite3 not installed. After installing it, I only had more issues.

Directory /etc/pihole does not exist. I changed this path in youtube.sh to what I'm guessing it should have been at /home/devhl/pihole/etc-pihole. This directory has gravity.db and some other things the script interacts with. I also created both black.list and blacklist.txt in this directory.

After making these changes and running the script again I get these errors

xargs: pihole: No such file or directory
sudo: pihole: command not found

This was because I am running pihole in a docker container.