Updgrade via GUI kills lm-sensors

[dec]

So I decided to click the 'upgrade' button in the web GUI (even though I'm already on v0.9) just to see if there was anything new (with the news of TwonkyMedia). Foolish, I know.


It appears to have killed my setup of lm-sensors in the System Information (phpsysinfo) page on my web GUI.


Is there a way to prevent this happening in the future?

[andrew]

That seems to be a bug. I'll see if I can fix that.

[dec]

I don't know if you've had a chance to look at this issue but I just wanted to let you know it's happened again with this latest update.


I updated to Squeezebox Server via the command line as per your instructions on the front page on the main site.


All that appears to be happening is the removal of the "lmsensors" line & the reinstatement "//hddtemp" quotes from the phpsysinfo page.

[andrew]

OK, I think I have this fixed. Do an update and it should be working.





yum clean all
yum update

[dec]

Hmm, well I updated as you suggested and it appears to have altered these lines again:


"%sensor_program = “lmsensors”;"


now reads:


$sensor_program = "";

and it has added the comment (//) for the line:

%hddtemp_avail = “suid”;

Or was the idea to prevent this from happening again, now that I've done the latest update?

I nearly forgot, but I also have this line:

$temperatureformat = "c";

That gets set back to:

$temperatureformat = "f";

when I update.

[andrew]

I realize it changed these lines back but I added some new code so it should still display the temp etc. Do the temp etc dispaly properlly? I'm not sure what to do about the f to c change. Seems like you will need change this after every upgrade. Maybe I can add a config file in the future that does this. Is that something your interested in?

[dec]

I did have to re-instate those lines to get the temperature to display, but it does display correctly.


If you can get the temperature setting to 'C' that would be great - so yes, I would be interested in that.


Thanks!

[andrew]

OK, I added this feature. You can go into system configuration and select your temp format now.




Please test his for me.



yum update



This will overwrite your config.php file again. You should not need to edit it. Everything should work automatically now.

[Moonbase]

Great idea to make °C/°F settable without config changes for us users in "Celsius countries"!


Lmsensors won't work on my old machine, but I use hddtemp. Will that changes also allow using hddtemp?


(Until my last update it always overwrote everything so my hddtemp values on the page were lost and needed to be re-modified into the page after each update.)

[dec]

Ok, I did the update as you suggested.


However I did have to re-instate the line:


$sensor_program = "lmsensors";


and remove the comment (//) from this line:

%hddtemp_avail = "suid";

to get the temps to display again. However I didn't have to change to 'c' again - that worked
great!

It's at least an improvement in one respect.

Thank you!