| 2012-07-06 19:45:24 |
Bence Lukács |
bug |
|
|
added bug |
| 2012-07-06 19:50:53 |
Bence Lukács |
psensor: status |
New |
Confirmed |
|
| 2012-07-06 22:06:16 |
Jean-Philippe Orsini |
bug task added |
|
psensor (Ubuntu) |
|
| 2012-07-06 22:06:34 |
Jean-Philippe Orsini |
psensor (Ubuntu): status |
New |
Confirmed |
|
| 2012-07-06 22:06:56 |
Jean-Philippe Orsini |
bug |
|
|
added subscriber jeanfi |
| 2012-07-06 22:56:16 |
Jean-Philippe Orsini |
attachment added |
|
LP1021900.debdiff https://bugs.launchpad.net/ubuntu/+source/psensor/+bug/1021900/+attachment/3215572/+files/LP1021900.debdiff |
|
| 2012-07-06 23:02:03 |
Jean-Philippe Orsini |
bug |
|
|
added subscriber Ubuntu Sponsors Team |
| 2012-07-10 08:49:21 |
Jean-Philippe Orsini |
tags |
|
quantal |
|
| 2012-07-10 08:49:41 |
Jean-Philippe Orsini |
tags |
quantal |
precise quantal |
|
| 2012-07-16 18:19:34 |
Micah Gersten |
psensor (Ubuntu): status |
Confirmed |
In Progress |
|
| 2012-07-16 18:19:34 |
Micah Gersten |
psensor (Ubuntu): assignee |
|
Micah Gersten (micahg) |
|
| 2012-07-16 18:19:40 |
Micah Gersten |
removed subscriber Ubuntu Sponsors Team |
|
|
|
| 2012-07-16 18:19:47 |
Micah Gersten |
nominated for series |
|
Ubuntu Precise |
|
| 2012-07-16 18:19:47 |
Micah Gersten |
bug task added |
|
psensor (Ubuntu Precise) |
|
| 2012-07-16 18:30:27 |
Micah Gersten |
psensor (Ubuntu): assignee |
Micah Gersten (micahg) |
|
|
| 2012-07-16 18:31:05 |
Micah Gersten |
psensor (Ubuntu Precise): importance |
Undecided |
Low |
|
| 2012-07-16 18:31:07 |
Micah Gersten |
psensor (Ubuntu): importance |
Undecided |
Low |
|
| 2012-07-16 18:34:28 |
Micah Gersten |
psensor (Ubuntu Precise): status |
New |
Triaged |
|
| 2012-07-16 18:34:42 |
Micah Gersten |
bug |
|
|
added subscriber Ubuntu Stable Release Updates Team |
| 2012-07-16 18:40:23 |
Launchpad Janitor |
psensor (Ubuntu): status |
In Progress |
Fix Released |
|
| 2012-07-16 19:14:15 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/psensor |
|
| 2012-07-16 20:29:35 |
Jean-Philippe Orsini |
attachment added |
|
LP1021900_KO.png https://bugs.launchpad.net/psensor/+bug/1021900/+attachment/3225364/+files/LP1021900_KO.png |
|
| 2012-07-16 20:32:15 |
Jean-Philippe Orsini |
attachment added |
|
LP1021900_OK.png https://bugs.launchpad.net/psensor/+bug/1021900/+attachment/3225366/+files/LP1021900_OK.png |
|
| 2012-07-17 08:01:15 |
Jean-Philippe Orsini |
attachment added |
|
LP1021900_OK.png https://bugs.launchpad.net/psensor/+bug/1021900/+attachment/3225793/+files/LP1021900_OK.png |
|
| 2012-07-17 08:02:44 |
Jean-Philippe Orsini |
attachment added |
|
lp1021900_precise.patch https://bugs.launchpad.net/psensor/+bug/1021900/+attachment/3225794/+files/lp1021900_precise.patch |
|
| 2012-07-17 08:23:42 |
Jean-Philippe Orsini |
description |
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 |
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
[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. |
|
| 2012-07-17 08:29:46 |
Jean-Philippe Orsini |
description |
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
[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. |
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. |
|
| 2012-07-17 08:43:07 |
Jean-Philippe Orsini |
description |
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. |
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... |
|
| 2012-07-19 16:38:47 |
Brian Murray |
psensor (Ubuntu Precise): status |
Triaged |
Fix Committed |
|
| 2012-07-19 16:38:50 |
Brian Murray |
bug |
|
|
added subscriber SRU Verification |
| 2012-07-19 16:38:56 |
Brian Murray |
tags |
precise quantal |
precise quantal verification-needed |
|
| 2012-07-19 17:15:38 |
Launchpad Janitor |
branch linked |
|
lp:~ubuntu-branches/ubuntu/precise/psensor/precise-proposed |
|
| 2012-08-08 14:55:09 |
Bence Lukács |
tags |
precise quantal verification-needed |
precise quantal verification-done |
|
| 2012-08-08 16:08:26 |
Colin Watson |
removed subscriber Ubuntu Stable Release Updates Team |
|
|
|
| 2012-08-08 16:09:11 |
Launchpad Janitor |
psensor (Ubuntu Precise): status |
Fix Committed |
Fix Released |
|
| 2012-09-11 08:08:14 |
Jean-Philippe Orsini |
psensor: status |
Confirmed |
Fix Released |
|
| 2012-09-15 22:33:42 |
Launchpad Janitor |
branch linked |
|
lp:debian/psensor |
|
