HenrikBengtsson / speedtest-cli-extras

:signal_strength: Tools to enhance the speedtest-cli network tools
120 stars 32 forks source link

update speedtest-cli call #16

Closed JulV94 closed 8 years ago

JulV94 commented 8 years ago

Avoid all the mess of $PATH variable when placed in a cronjob due to lack of /usr/local/bin for the speedtest-cli cmd. Avoid things like :

* * * * * PATH=$PATH:/usr/local/bin && export PATH && /path/to/speedtest-csv
HenrikBengtsson commented 8 years ago

Thanks. Unfortunately, we cannot make such assumptions about where people on different OSes have speedtest-cli installed.

A possible alternative to make speedtest-csv a bit smarter is for it to look for speedtest-cli in the same directory as itself and iff found there, then it could use that, and otherwise it falls back to what's on the $PATH.

HenrikBengtsson commented 8 years ago

Next release will look for speedtest-cli in the directory of speedtest-csv and only then in the PATH.