MycroftAI / enclosure-picroft

Mycroft interface for Raspberry Pi environment
https://mycroft.ai/documentation/picroft
GNU Lesser General Public License v3.0
804 stars 191 forks source link

check that $stage exists #116

Closed krisgesling closed 4 years ago

krisgesling commented 4 years ago

Was receiving the jq helper information when running through a new install.

Manually printing values during reruns of the script showed that $stage was empty, hence the setting of the variable by jq was failing.

This adds a check for $stage consistent with the other values being set.

forslund commented 4 years ago

Could the $stage be a typo and really supposed to be $setup_stage? the $stage variable is never set or used in the script as far as I can find...

krisgesling commented 4 years ago

Very good point, was still getting the jq info as $setup_stage was also set to an empty string so have left the test and updated the references to the correct variable name.

forslund commented 4 years ago

Gotcha, looks good. Merging