search

alectrocute / UnboundBL

🛑 DNSBL (adblock) on OPNsense with UnboundBL & Unbound DNS
62 stars 12 forks source link

Rename folders for naming consistency #5

Closed nekoprog closed 5 years ago

nekoprog commented 5 years ago

This will close #3 but not #4

alectrocute commented 5 years ago

I really appreciate the PR and your time. Seriously.

I made a commit to remove the +POST INSTALL file, which should fix the problem.

The naming convention is odd partly due to requirements with OPNsense’s plugin format.

With that said, this shouldn’t be a problem now. Refer again to the first line of this comment you’re currently reading.

nekoprog commented 5 years ago

Actually if you don't have to delete +POST INSTALL, just rename

/usr/local/opnsense/scripts/OPNsense/UnboundBL/UnboundBL.sh to /usr/local/opnsense/scripts/OPNsense/Unboundbl/UnboundBL.sh

chmod 755 /usr/local/opnsense/scripts/OPNsense/UnboundBL/data.sh to chmod 755 /usr/local/opnsense/scripts/OPNsense/Unboundbl/data.sh

it should be working fine too. Otherwise, you should rename the folder just like the PR. I found that the folder in github is not matching to +POST INSTALL, hence the error. Just need to rename the folder on local repo, delete all files on github and push again.