In Ubuntu 12.04 the Psensor shows the celsius degree badly. Both in the main window and in the indicator. For example if the temperature is 55 celsius degree it should be showed like this: 55°C not like this: 55 C, because it is very ugly.
I use Psensor 0.6.2.16
SRU information:
[Impact]
The degree Celcius symbol is missing.
Even if it is a real bug, it does not block any psensor feature usage, it is only cosmetic.
It will be nice to have it fixed in precise which is a LTS but should be handled with
a very very low priority. SRU team members are certainly more useful patch to check and
upload than this one.
The fix is trivial, it simply adds the celcius degree sign before the 'C'.
[Test Case]
Here is a screenshot showing the bug: https://bugs.launchpad.net/psensor/+bug/1021900/+attachment/3225364/+files/LP1021900_KO.png
The missing degree sign is emphasized in the red circles
To reproduce:
- 1 - ubuntu precise must be installed in a real computer and not in a virtual machine.
VM does not allow to retrieve thermal sensors of the computer. The hardware
must have at least one thermal sensor supported by lmsensor.
- 2 - install psensor
- 3 - starts the sensor detection script and follow its instruction: sudo sensors-detect
- 4 - reboot the computer to ensure that the needed kernel module are loaded
- 5 - starts 'psensor', wait a couple of seconds until the sensors measures are retrieved
- 6 - check that all celcius units displayed in the main window have the degree sign
- 7 - open the psensor application indicator and checks the degree sign
You can see the correct display in this screenshot:
https://bugs.launchpad.net/psensor/+bug/1021900/+attachment/3225793/+files/LP1021900_OK.png
A unit test has been added for this bug in the maintream dev branch, but it involves modification of the build files that I am not yet confortable with. So, I believe it is safer to do a manual check than port the unit test to the patch for this fix:
http://wpitchoune.net/svnpub/psensor/trunk/tests/test_psensor_type_to_unit_str.c
and http://wpitchoune.net/svnpub/psensor/trunk/tests/test_psensor_value_to_str.c
[Regression Potential]
The fix is trivial, the unit string are not dynamicaly allocated nor translatable, I don't expect any regression.
The fix has been commited upstream in mainstream dev branch few months ago and nobody
raised a regression bug. Mainstream dev branch has been tested on both Ubuntu and Debian.
The only probleme with this fix that I can imagine is that the used default font does not have the glyph corresponding to the degree sign. It might lead to a corrupted display but most used font certainly has the degree sign...
Already fixed upstream few months ago in the development branch (will be in the future 0.7.0 release).
You can give it a try using ppa:jfi/
Thanks for the report.