search

BrewPi / brewpi-tools

Various tools to setup/update/configure BrewPi
GNU General Public License v3.0
39 stars 46 forks source link

Added required file check #12

Closed vanosg closed 10 years ago

vanosg commented 11 years ago

Current exception will catch update-this-repo.sh not existing as a ProcessError, and display the error "script not up to date, please re-run updater.py", assuming the only error that this would catch is an update error. If the user moves this file outside of the utils dir for some reason, or copies it into the brewpi-script dir from an external brewpi-tools repo (and not the other required files), this error would be misleading. This patch checks to see if required file exist in same dir and suggest a solution.

elcojacobs commented 11 years ago

The extra check is a good idea, but the message is wrong:

The files update-this-repo.sh and updater.py are part of the brewpi-tools repo, and not /utils. The message should be: please keep updater.py in the brewpi-tools git repository.