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.

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

Update update_vendors.sh #221

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dougie23fresh
Copy link

-LO will follow redirects

-L in case the page has moved (3xx response) curl will redirect the request to the new address -o output to a file instead of stdout (usually the screen). In your case the o flag is redundant since the output is piped to bash (for execution) - not to a file.

-LO will follow redirects

-L in case the page has moved (3xx response) curl will redirect the request to the new address
-o output to a file instead of stdout (usually the screen). In your case the o flag is redundant since the output is piped to bash (for execution) - not to a file.
@andriymoroz
Copy link

upvoting...

@jokob-sk
Copy link

@andriymoroz

The main fork is pretty much dead, however, check out this comment: #220 (comment)

@herbiegreenfield
Copy link

Upvoting.
I just manually edited update_vendors.sh

only change was that each line now uses -LO.

sudo curl $1 -# -LO http://standards-oui.ieee.org/iab/iab.csv

sudo curl $1 -# -LO http://standards-oui.ieee.org/iab/iab.txt

sudo curl $1 -# -LO http://standards-oui.ieee.org/oui28/mam.csv

sudo curl $1 -# -LO http://standards-oui.ieee.org/oui28/mam.txt

sudo curl $1 -# -LO http://standards-oui.ieee.org/oui36/oui36.csv

sudo curl $1 -# -LO http://standards-oui.ieee.org/oui36/oui36.txt

sudo curl $1 -# -LO http://standards-oui.ieee.org/oui/oui.csv

sudo curl $1 -# -LO http://standards-oui.ieee.org/oui/oui.txt

@jokob-sk
Copy link

jokob-sk commented May 8, 2023

FYI the original report wasn't updated for 2 years. Check out the latest comment here #221

@herbiegreenfield
Copy link

Thank you, yes this latest comment #221 gave me the info to modify BY ADDING “O” to flag -LO.

I landed into this from an old Reddit Post and subsequently installed, trouble shot, then uninstalled after seeing the forks. That led to a lot more troubleshooting too. :) for the server direct install on Raspberrian leiweibau fork which needed tweaks for python3 and for python libraries I was missing. Then log files are missing for php and the symlinks step fails too.

So far my install is running and I just need to prune down the hundred+ legacy devices imported from past years of PiHole. I suspect that dynamic IP addressing plays a part but I have a mess to untangle for PiAlert and PiHole to align logical common names to MAC addresses, and the moving IP addresses. Example: my smart Tv is labeled as a MacBook but vendor is Vizio. And there are a hundred+ untitled entries.

Thanks so much for the incredible improvements and contributions!!!

@leiweibau
Copy link

I think we active maintainers only do this because we think the "tool" "pucherot/Pi.Alert" is great and that this software deserves to be maintained to stay great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants