Script fails post merge of Dockerized PR

Sennevds / system_sensors

Logging of system sensor specific for the RPI and sending them to a MQTT broker

MIT License

361 stars 112 forks source link

Script fails post merge of Dockerized PR #147

Closed mmattel closed 1 year ago

mmattel commented 1 year ago

When running system_sensors via systemd, after the merge of @emkookmer #110 (Dockerize (revised)), you get following error making the script fail and unusable.

system_sensors/src/system_sensors.py", line 167, in get_host_model
model = f'{deviceManufacturer} {deviceNameDisplay}'
NameError: name 'deviceManufacturer' is not defined

@emkookmer pls fix

mmattel commented 1 year ago

Confirmed working again with the above commit, thanks 👍