search

adamoutler / HassOSConfigurator

A set of Add-Ons to configure HassOS For various platforms and development purposes.
184 stars 44 forks source link

Run: run.sh line 59 logOption is unbound #6

Closed yelvert closed 2 years ago

yelvert commented 3 years ago

It looks like the logOption variable is only ever getting defined if the Retain old logs in /config/startup/logs/ instead of deleting old log config option is true. When that option is false logOption should be set to "" when that config option is false.

adamoutler commented 3 years ago

Is this still valid?

yelvert commented 3 years ago

Yeah, see line 21 of that run.sh script, the logOption variable only gets created if the test passes, which means if it doesn't pass then anything that references that variable later will throw an error.

adamoutler commented 2 years ago

fixed. sorry it took so long.